well, upstart descriptions would be a nice thing to have imo, but first
we should get the package building/working again, which doesn't need an
FFe being of bugfix nature.

Looking at the patch I've got the following comments:
1) the "function" keyword is a bashism, however functions are POSIX (and work 
with dash), so simplest way would be to just remove the function keyword
2) bash is Essential: Yes, so it won't add any "weight" to mini-httpd (despite 
that it loads slower and hence adds to boot time). This also means that it 
doesn't need to be in Depends. (however fixing 1 means to not need bash at all).
3) why do you rename putline as well? I might be wrong, but I don't recall that 
having been added to POSIX and a quick grep over /usr/include doesn't result in 
any matches.
4) Personally I'm in favor of trying to keep the patches down to a minimum 
(we'll otherwise have to redo them for the next merge), so I'd personally avoid 
bumping standards-version, debhelper version or adding the README about patched 
package (heh, finally I guess the last point is something every Ubuntu 
Developer knows by heart anyways *g*).

As 1) is fixed with 1.19-9.1, can you put the fix for getline on top of 
1.19-9.1? Then I'd be happy to sponsor the upload.
Thanks,
    Stefan.

-- 
mini-httpd sysv init script now requires bash
https://bugs.launchpad.net/bugs/403286
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to