Merge lp://qastaging/~tom-leiming/udisks/bug435192 into lp://qastaging/ubuntu/precise/udisks

Proposed by Ming Lei
Status: Rejected
Rejected by: Martin Pitt
Proposed branch: lp://qastaging/~tom-leiming/udisks/bug435192
Merge into: lp://qastaging/ubuntu/precise/udisks
Diff against target: 5256 lines (+747/-906)
22 files modified
debian/changelog (+5/-12)
debian/control (+1/-2)
debian/local/apport-hook.py (+2/-2)
doc/html/Adapter.html (+16/-16)
doc/html/Device.html (+233/-233)
doc/html/Expander.html (+17/-17)
doc/html/Port.html (+15/-15)
doc/html/UDisks.html (+82/-82)
doc/html/index.html (+3/-3)
doc/html/ix01.html (+236/-236)
doc/html/license.html (+2/-2)
doc/html/ref-dbus.html (+2/-2)
doc/html/tools-fileformats.html (+2/-2)
doc/html/udisks-daemon.8.html (+7/-7)
doc/html/udisks-tcp-bridge.1.html (+6/-6)
doc/html/udisks.1.html (+7/-7)
doc/html/udisks.7.html (+10/-10)
doc/html/udisks.devhelp (+0/-250)
policy/org.freedesktop.udisks.policy (+82/-0)
src/device.c (+17/-0)
src/marshal.c (+1/-1)
tools/udisks-marshal.c (+1/-1)
To merge this branch: bzr merge lp://qastaging/~tom-leiming/udisks/bug435192
Reviewer Review Type Date Requested Status
Martin Pitt Disapprove
Review via email: mp+119457@code.qastaging.launchpad.net

Description of the change

Add ro flag to mount if the disk is readonly

To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

This will be/is fixed properly in ntfs-3g instead, see the bug. Thanks!

review: Disapprove

Unmerged revisions

36. By Ming Lei

Apply patch to fix mounting readonly block device

35. By Martin Pitt

* debian/tests/control: Add mtools dependency for VFAT test. (LP: #976269)
* debian/tests/control: Add policykit-1 dependency. (LP: #976266)
* Add 00git_rts_bpp_sdcard.patch: Mark rts_bpp devices as SD card readers.
  Patch also committed upstream. (LP: #971876)
* debian/local/apport-hook.py: Fix syntax to also work with Python 3.
* debian/control: Change the "cryptsetup" suggests to a "cryptsetup-bin"
  recommends, so that LUKS functionality works by default.

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: