Comment 10 for bug 196778
Revision history for this message
| Emilio Pozuelo Monfort (pochu) wrote | #10 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
a60fb26
(Get the code!)
Hi Nicolas,
Two comments for your debdiff:
- You have totally broken the clean target. The clean target in your debdiff does nothing. You don't need to comment this things:
# dh_testdir
# dh_testroot
# rm -f build-stamp configure-stamp
# $(MAKE) clean
# dh_clean
- I think adding another .desktop file only for KDE is suboptimal... Another way would be to patch the desktop file to use su-to-root. Although that would add a dependency on menu, so let's leave your change, which is good too. I hope we have a su-to-root freedesktop wrapper provided in xdg-utils.