Bug 338753 – Version bump for server-www/lighttpd 1.4.26 -> 1.4.28

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Nikhil Sethi 2010-09-26 04:40:41 UTC]

Version bump for lighttpd from 1.4.26 to 1.4.28

I have tested the ebuild (lighttpd-1.4.26.ebuild renamed to lighttpd-1.4.28.ebuild) using

ebuild /usr/local/portage/www-servers/lighttpd/lighttpd-1.4.28.ebuild test

Installed and run on x86 platform.


Reproducible: Always

[Comment 1 Andrii Yurchuk 2010-09-27 19:39:26 UTC]

A new feature was introduced in 1.4.27 - support for libev. This should be taken into consideration when bumping.

[Comment 2 Nikhil Sethi 2010-09-27 20:15:49 UTC]

Created attachment 248828 [details]
ebuild with support for libev

Good catch. Attached ebuild file adds a libev USE flag to enable libev support.

[Comment 3 Andrii Yurchuk 2010-09-29 18:02:47 UTC]

(In reply to comment #2)
> Created an attachment (id=248828) [details]
> ebuild with support for libev
> 
> Good catch. Attached ebuild file adds a libev USE flag to enable libev support.
> 

It seems to me that the ebuild for .28 needs more work. At least because of the wrong doc files paths. But what's more important -- .28 has new approach to storing config files and I think it should be reflected in the ebuild and the config file structure for lighttpd in Gentoo should also be changed.
I'll try to find some time to work on it.

[Comment 4 Andrii Yurchuk 2010-10-01 21:45:07 UTC]

Here's the updated .28 ebuild: http://github.com/Ch00k/minute/tree/master/www-servers/lighttpd/
The config files may still need some work, but Lighttpd works with the current ones right after installation.

[Comment 5 Nathan 2010-10-18 16:57:15 UTC]

I'd be willing to test this if we can get it into portage.

[Comment 6 Markos Chandras (RETIRED) 2010-11-02 16:52:53 UTC]

Please provide a diff over the current ebuild so we can spot the changes more easily

[Comment 7 Andrii Yurchuk 2010-11-03 09:31:24 UTC]

Created attachment 253017 [details, diff]
diff of 1.4.28 over 1.4.26-r1

[Comment 8 Markos Chandras (RETIRED) 2010-11-03 09:35:24 UTC]

Looks good but why did you remove all these parts from pkg_postinst() ?

[Comment 9 Andrii Yurchuk 2010-11-03 09:46:09 UTC]

Well, I don't think they are still needed. As one of them say, "As of lighttpd-1.4.22, spawn-fcgi is provided by the separate package". We do not have 1.4.22 in the tree anymore, and I do not think anybody has it installed. But if you still think that we need these ewarn's - we can leave them intact.

[Comment 10 Markos Chandras (RETIRED) 2010-11-12 21:49:26 UTC]

Created attachment 254135 [details, diff]
proposed patch

This is my proposed diff over 1.4.26. Alex any comments on that? This also addresses bug 312421 too.

[Comment 11 Markos Chandras (RETIRED) 2010-11-12 22:55:47 UTC]

Created attachment 254145 [details, diff]
revised diff

[Comment 12 Alex Alexander (RETIRED) 2010-11-15 15:50:36 UTC]

*** Bug 345631 has been marked as a duplicate of this bug. ***

[Comment 13 Markos Chandras (RETIRED) 2010-11-15 17:31:39 UTC]

The new ebuild ( as per attached diff ) will hit portage in 72 hours. If you spot any mistake or regression over the current ebuild please let us know

Thanks

[Comment 14 Markos Chandras (RETIRED) 2010-11-15 17:33:52 UTC]

Created attachment 254403 [details, diff]
latest revision

[Comment 15 Markos Chandras (RETIRED) 2010-11-18 15:13:55 UTC]

Bumped. Thank you all 

[Comment 16 Andrii Yurchuk 2010-11-18 16:08:55 UTC]

Shouldn't line #98 be a comment?

[Comment 17 Markos Chandras (RETIRED) 2010-11-18 16:14:24 UTC]

(In reply to comment #16)
> Shouldn't line #98 be a comment?
> 

Yes. Fixed. Thank you