Bug 96936 – Ebuild change: shttpd (new version, change in packagement)

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description lzap 2005-06-24 03:52:37 UTC]

I have created new version of the ebuild, that compiles versions >= 1.24 because 
the author of this small webserver changed the package (no he uses tarball 
instead of single .c file).

Reproducible: Always
Steps to Reproduce:
1.
2.
3.




I am including patches for ebuild, xinetd config, init.d config and the new 
configuration file (shttpd now uses /etc/shttpd/shttpd.conf file).

Including also already-patched versions for those who do not like patches :-)

[Comment 1 lzap 2005-06-24 03:53:42 UTC]

Its a www-servers/shttpd.

[Comment 2 lzap 2005-06-24 03:55:03 UTC]

Created attachment 61831 [details, diff]
ebuild 1.21 --> 1.25 diff

[Comment 3 lzap 2005-06-24 03:55:36 UTC]

Created attachment 61832 [details, diff]
files/shttpd.conf patch to v1.25

[Comment 4 lzap 2005-06-24 03:56:44 UTC]

Created attachment 61833 [details, diff]
files/shttpd.confd patch to v1.25

confd (not conf) - sorry, because there is also "conf" file

[Comment 5 lzap 2005-06-24 03:57:03 UTC]

Created attachment 61834 [details, diff]
files/shttpd.xinetd patch to v1.25

[Comment 6 lzap 2005-06-24 03:57:44 UTC]

Created attachment 61835 [details]
shttpd now uses config file, this is gentoo-ready config

files/shttpd.conf

[Comment 7 lzap 2005-06-24 03:58:49 UTC]

Created attachment 61836 [details]
Patched version 1.25 (if you do not like patch :-)

[Comment 8 lzap 2005-06-24 03:59:05 UTC]

Created attachment 61837 [details]
Patched version 1.25 (if you do not like patch :-)

[Comment 9 lzap 2005-06-24 03:59:22 UTC]

Created attachment 61838 [details]
Patched version 1.25 (if you do not like patch :-)

[Comment 10 lzap 2005-06-24 04:01:23 UTC]

I am not sure about braking lines in ebuilds with "\".

It works but there are much useless whitespace - please fix it.

  opts="-DCONFIG=\"/etc/shttpd/shttpd.conf\" \
                        -DWITH_SSL ${PN}.c -o ${PN} -lcrypto -lssl"

[Comment 11 Aaron Walker (RETIRED) 2005-06-27 18:46:03 UTC]

Please use the unified diff format (diff -u) in the future.  No need to post
patched versions, as just the diffs will do.  This is on my todo list for
tonight at work so it should be done in the morning.

[Comment 12 Aaron Walker (RETIRED) 2005-06-27 18:48:54 UTC]

oops and I forgot to mention.  Always use text/plain for text files (patch is
fine for patches), otherwise it's not possible to view it in the browser.

[Comment 13 Aaron Walker (RETIRED) 2005-06-28 06:56:20 UTC]

Did you try testing it at all?

There seems to be something b0rked with the config file code.  Unless I remove
the config file, it dies every time with:

].Each line in config file must be in format '<var> <value>'

It does this regardless of config (yours or the one distributed by uptstream). 
I don't have the time now to look into it, but needless to say until I get/write
a patch or he releases a new version that fixes it, it's not going into portage.

[Comment 14 lzap 2005-06-28 09:49:56 UTC]

Got the bug!

The conf file was created under Win32 with Windows-like linefeeds. Please fix it
using Vim or recode, thank you. Will notice the author.

[Comment 15 Aaron Walker (RETIRED) 2005-06-30 21:49:03 UTC]

In CVS.  Thanks Lukas.