Bug 5162

Summary: Canonical has broken Debian's install_initd
Product: ThinLinc Reporter: Pierre Ossman <ossman>
Component: Server OSAssignee: Samuel Mannehed <samuel>
Status: CLOSED FIXED    
Severity: Normal CC: aaron
Priority: P2 Keywords: prosaic, samuel_tester
Version: trunk   
Target Milestone: 4.3.0   
Hardware: PC   
OS: Unknown   
Acceptance Criteria:

Description Pierre Ossman cendio 2014-05-23 14:00:31 CEST
Debian 7 and later has a completely broken install_initd thanks to the same crack team at Canonical that caused bug 4205 and bug 4439 (and forced us to do bug 4546). This time though they managed to screw Debian over, and not themselves.

Upstream bug here:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=743903

Bad commit here:

http://anonscm.debian.org/gitweb/?p=collab-maint/lsb.git;a=commitdiff;h=f4ed7f08600d633c3daba9f494997f1c35555aed

StringIO.StringIO and io.StringIO do not have identical API.
Comment 1 Pierre Ossman cendio 2014-05-27 10:26:57 CEST
Start by reporting more information to Debian and updating our platform specific notes with a workaround. If Debian fails to take action we'll start looking into changing the product.
Comment 2 Aaron Sowry cendio 2014-05-27 11:15:29 CEST
Additional info has been added to Debian's bug report.
Comment 3 Aaron Sowry cendio 2014-06-03 12:34:21 CEST
Not convinced the maintainers will fix this within any reasonable timeframe, the thing's a mess. We should at least write up something in Platform Specific Notes in the mean time.
Comment 4 Aaron Sowry cendio 2014-06-09 10:36:35 CEST
(In reply to comment #3)
> Not convinced the maintainers will fix this within any reasonable timeframe,
> the thing's a mess. We should at least write up something in Platform Specific
> Notes in the mean time.

I stand corrected, looks like it's been fixed upstream.
Comment 5 Samuel Mannehed cendio 2014-07-15 11:04:04 CEST
I installed up both a debian 7.5 and a 7.6, installed ThinLinc and verified that the bug is indeed fixed.
Comment 6 Samuel Mannehed cendio 2014-07-24 13:52:50 CEST
Reopening, not working in Debian 8:

2014-07-24 12:32:00,164: Output (stderr):
2014-07-24 12:32:00,164:     Traceback (most recent call last):
2014-07-24 12:32:00,164:       File "/usr/lib/lsb/install_initd", line 46, in <module>
2014-07-24 12:32:00,165:         headers = initdutils.scan_initfile(initfile)
2014-07-24 12:32:00,165:       File "/usr/lib/lsb/initdutils.py", line 79, in scan_initfile
2014-07-24 12:32:00,165:         inheaders = RFC822Parser(strob=headerlines)
2014-07-24 12:32:00,165:       File "/usr/lib/lsb/initdutils.py", line 24, in __init__
2014-07-24 12:32:00,165:         fileob = StringIO(strob)
2014-07-24 12:32:00,165:     TypeError: initial_value must be unicode or None, not str
2014-07-24 12:32:00,166: Failed to install service 'vsmserver'
2014-07-24 12:32:10,729: tl-setup aborted by user
Comment 7 Samuel Mannehed cendio 2014-07-24 20:02:20 CEST
> If you have further comments please address them to 743903@bugs.debian.org,
> and the maintainer will reopen the bug report if appropriate.

Done.
Comment 9 Samuel Mannehed cendio 2014-07-25 14:21:36 CEST
Platform-specific note with workaround added in revision 29222.
Comment 10 Samuel Mannehed cendio 2014-09-24 14:42:15 CEST
Looks good.