Merge lp://qastaging/~xmacex/ubuntu/raring/modemmanager/ignore-arduino-lilypad-usb into lp://qastaging/ubuntu/raring/modemmanager

Proposed by Mace
Status: Work in progress
Proposed branch: lp://qastaging/~xmacex/ubuntu/raring/modemmanager/ignore-arduino-lilypad-usb
Merge into: lp://qastaging/ubuntu/raring/modemmanager
Diff against target: 27 lines (+9/-0)
2 files modified
debian/changelog (+6/-0)
src/77-mm-usb-device-blacklist.rules (+3/-0)
To merge this branch: bzr merge lp://qastaging/~xmacex/ubuntu/raring/modemmanager/ignore-arduino-lilypad-usb
Reviewer Review Type Date Requested Status
James Page Needs Fixing
Ubuntu branches Pending
Review via email: mp+152710@code.qastaging.launchpad.net

Description of the change

Hi, i added Arduino LilyPad USB as one of the devices modemmanager should ignore. This is my first ever bugfix for Ubuntu, so i hope i made everything correctly after reading the bugfixing documentation. Cheers!

To post a comment you must log in.
Revision history for this message
James Page (james-page) wrote :

Hi Mace

Thanks for taking the time to propose this fix for Ubuntu.

As this is your first bugfix now is a great time to learn about how the 'source format' of a Debian package works.

This package is source format 3.0 (quilt) (see debian/source/format); as a result when we do fixes in packaging, they are applied as patches to the original source code as part of the package build process, rather than applying the fixes directly to the upstream source code.

This package already has a number of such fixes - see debian/patches/*.

Have a read of - http://developer.ubuntu.com/packaging/html/patches-to-packages.html - it will walk you through how to prepare your fix as a quilt patch.

Cheers

James

Revision history for this message
James Page (james-page) :
review: Needs Fixing

Unmerged revisions

37. By Mace

src/77-mm-usb-device-blacklist.rules: included Arduino LilyPad USB as a device modemmanager should ignore (LP: #1153632)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: