Bug 697523 - The old SysV initscript should be packaged into subpackage when replaced with systemd one
Summary: The old SysV initscript should be packaged into subpackage when replaced with...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: dbus
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: David Zeuthen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: SysVtoSystemd
TreeView+ depends on / blocked
 
Reported: 2011-04-18 14:09 UTC by Jóhann B. Guðmundsson
Modified: 2013-03-06 04:07 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-08-03 21:54:50 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
I think this patch should satisfie the packaging guidelines and drops sysv support (1.42 KB, patch)
2011-06-30 18:41 UTC, Jóhann B. Guðmundsson
no flags Details | Diff
I think this patch should satisfie the packaging guidelines and drops sysv support (1.42 KB, patch)
2011-06-30 18:44 UTC, Jóhann B. Guðmundsson
no flags Details | Diff
Spec file patch for dbus (1.42 KB, patch)
2011-07-26 22:01 UTC, Jóhann B. Guðmundsson
no flags Details | Diff

Description Jóhann B. Guðmundsson 2011-04-18 14:09:24 UTC
Description of problem:

The old SysV initscript should be package into seperate subpackage when replaced with a native systemd one to avoid confusion amongs end users since systemd will use the native systemd service file when it exist by default thus rendering the old sysv obsolete and keeping it around will only confuse users that for one reason or another are editing the sysv init script.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Jóhann B. Guðmundsson 2011-06-27 12:07:45 UTC
What's the current status on this?

We need this in rawhide the sooner the better.

https://fedoraproject.org/wiki/Packaging:Guidelines:Systemd
https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Systemd
http://fedoraproject.org/wiki/Packaging:Tmpfiles.d

Comment 2 Jóhann B. Guðmundsson 2011-06-30 18:41:50 UTC
Created attachment 510733 [details]
I think this patch should satisfie the packaging guidelines and drops sysv support

Comment 3 Jóhann B. Guðmundsson 2011-06-30 18:44:13 UTC
Created attachment 510735 [details]
I think this patch should satisfie the packaging guidelines and drops sysv support

Minor fix to the trigger run version

Comment 4 Jóhann B. Guðmundsson 2011-07-15 15:07:21 UTC
What's the current status of you are one of few that is not package
native systemd unit in the Base group correctly?

If you dont have the time to fix this should I try to find a proven packager to
do the work for you? 

https://fedoraproject.org/wiki/Packaging:Guidelines:Systemd
https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Systemd
https://fedoraproject.org/wiki/Packaging:Tmpfiles.d

Comment 5 Colin Walters 2011-07-18 18:31:19 UTC
(In reply to comment #4)
> What's the current status of you are one of few that is not package
> native systemd unit in the Base group correctly?
> 
> If you dont have the time to fix this should I try to find a proven packager to
> do the work for you? 

Frankly I don't really believe upgrades like this have ever worked reliably; if you've tested the code at all, feel free to commit.

(Did you test it?)

Comment 6 Jóhann B. Guðmundsson 2011-07-18 19:06:44 UTC
I tested building it that's about it I'm not a proven packager however I shall see if I cant find one tomorrow that will review and commit the changes since you are busy.

Comment 7 Jóhann B. Guðmundsson 2011-07-26 22:01:29 UTC
Created attachment 515379 [details]
Spec file patch for dbus

Note this needs to be fixed before thursday thanks.

Comment 8 David Zeuthen 2011-08-03 13:38:06 UTC
Uhm, comment 5 asked for whether you had tested it and you replied in comment 6 that you've only _built_ it. It is not reasonable to then ask for it to be committed if you haven't even tested it.

Comment 9 Jóhann B. Guðmundsson 2011-08-03 13:49:37 UTC
Well it was broken the other patch which is why I came up with a new one ( comment 7 ) and the only reason I'm providing spec patches is to speed up the process it's been ca 4 months ( yup this report is getting 4 months old ) which should have been a sufficient time to find the time to clean the spec file and do an 10 - 20 minute job anyway just review the patch and commit or come up with your own we kinda had wanted this to be done last thursday since we wanted the live alpha compose to be legacy sysv init script free.

Comment 10 David Zeuthen 2011-08-03 13:59:42 UTC
No, no, no. Please realize that it's never a "10-20 minute job" when it comes to something as critical as the D-Bus packages. If they are broken, the system likely won't boot (at least not into any useful state) and you'll ruin the day for a lot of people on Rawhide by pushing them out. This means that you need to test the packages *thoroughly* in addition to just reviewing the changes made to them.

If you had replied something like "yup, I tested the packages and both the upgrade as well as a subsequent reboot worked fine" then I'd more much more likely to commit the patch (after testing it myself, of course).

Comment 11 David Zeuthen 2011-08-03 14:01:41 UTC
Lennart: would you mind skimming through the proposed patch in comment 7 and see if it's correct? Thanks!

Comment 12 Lennart Poettering 2011-08-03 14:05:57 UTC
Looks good!

Comment 13 David Zeuthen 2011-08-03 21:54:50 UTC
OK, I tested the patch and it works both after the update and also after a reboot. Built for f16 here

 http://koji.fedoraproject.org/koji/taskinfo?taskID=3251000


Note You need to log in before you can comment on or make changes to this bug.