Merge lp://qastaging/~ilidrissi.amine/update-notifier/renaming into lp://qastaging/update-notifier/ubuntu

Proposed by Mohamed Amine Ilidrissi
Status: Rejected
Rejected by: Brian Murray
Proposed branch: lp://qastaging/~ilidrissi.amine/update-notifier/renaming
Merge into: lp://qastaging/update-notifier/ubuntu
Diff against target: 652 lines (+89/-83)
33 files modified
HOOKS (+4/-4)
README (+2/-2)
README.testing (+3/-3)
autogen.sh (+1/-1)
configure.in (+3/-3)
data/Makefile.am (+4/-4)
data/apt_check.py (+4/-4)
data/notify-reboot-required (+1/-1)
data/update-manager-daemon.desktop.in (+4/-4)
data/update-motd-updates-available (+1/-1)
debian/20-cpu-checker (+1/-1)
debian/90-updates-available (+1/-1)
debian/98-reboot-required (+1/-1)
debian/changelog (+5/-2)
debian/control (+10/-7)
debian/update-manager-daemon-common.dirs (+1/-1)
debian/update-manager-daemon-common.install (+2/-2)
debian/update-manager-daemon-common.links (+2/-2)
debian/update-manager-daemon.install (+5/-5)
po/POTFILES.in (+3/-3)
src/Makefile.am (+9/-9)
src/avahi.c (+1/-1)
src/clipboard.c (+1/-1)
src/crash.c (+1/-1)
src/gdu.c (+4/-4)
src/hook-remove.c (+1/-1)
src/hooks.c (+1/-1)
src/reboot.c (+1/-1)
src/release.c (+1/-1)
src/update-manager-daemon.c (+6/-6)
src/update-manager-daemon.h (+2/-2)
src/update.c (+2/-2)
ui/Makefile.am (+1/-1)
To merge this branch: bzr merge lp://qastaging/~ilidrissi.amine/update-notifier/renaming
Reviewer Review Type Date Requested Status
Brian Murray Disapprove
Review via email: mp+32661@code.qastaging.launchpad.net

Description of the change

This branch renames update-notifier to update-manager-daemon.

To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

update-notifier does more (now) than just call update-manager, subsequently I think renaming it wholesale is inappropriate.

review: Disapprove

Unmerged revisions

591. By Mohamed Amine Ilidrissi

Renamed update-notifier to update-manager-daemon (LP: #399591)

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: