Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 658470

Summary: www-apps/webdavcgi-1.1.2: version bump (and Homepage change)
Product: Gentoo Linux Reporter: Alexander Weber <web.alexander>
Component: Current packagesAssignee: Christian Affolter <c.affolter>
Status: RESOLVED FIXED    
Severity: normal CC: dev-zero, jstein, proxy-maint
Priority: Normal Keywords: PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://bugs.gentoo.org/show_bug.cgi?id=674772
https://github.com/gentoo/gentoo/pull/10826
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 740908    

Description Alexander Weber 2018-06-19 08:59:46 UTC
The current package homepage http://webdavcgi.sourceforge.net/ does forward to https://danrohde.github.io/webdavcgi/
On the github page there is a 1.1.2 release
Please bump and update the www-apps/webdavcgi ebuild
Comment 1 Christian Affolter 2018-06-26 16:46:27 UTC
I'm sorry, but I'm currently too busy to bump and test a new WebDAV CGI release. You're more than welcome to step in if you like.

According to the upgrade notes [1] a new IO::Compress::Brotli [2] ebuild would be required as well.

[1] https://danrohde.github.io/webdavcgi/doc.html#upgrade
[2] https://metacpan.org/pod/distribution/IO-Compress-Brotli/lib/IO/Compress/Brotli.pm
Comment 2 Alexander Weber 2019-01-07 10:13:51 UTC
Current stable does not work with stable perl 5.26 because of unescaped left brace

Issue seems to be fixed in upstream in https://github.com/DanRohde/webdavcgi/commit/04e79b7ecbaf3aae5ab813cd4fc0a009c72b1580
Comment 3 Christian Affolter 2019-01-17 13:39:45 UTC
Dear Alexander,

thanks a lot for the report and the proposed regex patch. I've finally found some time to rework on the ebuild and update it to the latest upstream version.

The PR is available on GitHub:
https://github.com/gentoo/gentoo/pull/10826
Comment 4 Alexander Weber 2020-01-10 12:03:38 UTC
Any chance to get the PR in portage? I use the www-apps/webdavcgi-1.1.2 compiled using this ebuild since ~1 year without issues
Comment 5 Larry the Git Cow gentoo-dev 2020-02-04 10:43:31 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=30d6e9e94eb2a0b2a72778d9fcda422678eed08e

commit 30d6e9e94eb2a0b2a72778d9fcda422678eed08e
Author:     Kent Fredric <kentnl@gentoo.org>
AuthorDate: 2020-02-04 10:38:40 +0000
Commit:     Kent Fredric <kentnl@gentoo.org>
CommitDate: 2020-02-04 10:40:09 +0000

    dev-perl/IO-Compress-Brotli: Add w/ version 0.4.1
    
    Required as a dependency for bumping www-apps/webdavcgi to 1.1.2
    
    Change suggested by Christian Affolter, but reworked from scratch
    to incorporate unbundling of brotli, and to employ proper versions,
      and EAPI7.
    
    Bug: https://bugs.gentoo.org/658470
    Bug: https://github.com/gentoo/gentoo/pull/10826
    Package-Manager: Portage-2.3.86, Repoman-2.3.20
    Signed-off-by: Kent Fredric <kentnl@gentoo.org>

 .../IO-Compress-Brotli-0.4.1.ebuild                | 28 +++++++++++++
 dev-perl/IO-Compress-Brotli/Manifest               |  1 +
 .../IO-Compress-Brotli-0.004001-unbundle.patch     | 49 ++++++++++++++++++++++
 dev-perl/IO-Compress-Brotli/metadata.xml           | 13 ++++++
 4 files changed, 91 insertions(+)
Comment 6 Christian Affolter 2020-09-11 12:48:31 UTC
Is there anything missing to get this merged? It would be great if some proxy maintainer could have a look at the mentioned PR (https://github.com/gentoo/gentoo/pull/10826)

Thanks!
Comment 7 Larry the Git Cow gentoo-dev 2021-01-05 08:39:57 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d1f48c31817d15862ca180e32f4449117c49e87a

commit d1f48c31817d15862ca180e32f4449117c49e87a
Author:     Christian Affolter <c.affolter@purplehaze.ch>
AuthorDate: 2019-01-13 15:39:08 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2021-01-05 08:39:13 +0000

    www-apps/webdavcgi: Version bump to 1.1.2
    
    Add fix unescaped braces patch.
    This patch fixes the unescaped brace regex warnings, which break the
    usage of WebDAV CGI with Perl >= 5.26 as reported by Alexander Weber
    within #674772 and #658470.
    
    It is based on:
    https://github.com/DanRohde/webdavcgi/commit/04e79b7ecbaf3aae5ab813cd4fc0a009c72b1580
    
    Apart from the version bump, the following enhancements were made:
    * Use EAPI 7
    * Updated home page and external links of the WebDAV CGI project.
    * Depend on dev-perl/IO-Compress-Brotli, which is required for 1.1.2.
    * Fix unescaped braces (addresses #674772 and #658470)
    * Add note regarding systemd and setuid/setgid webdavwrapper
    * Using EROOT instead of ROOT for displaying installation paths
    * Removed no longer used eutils dependency
    
    Closes: https://bugs.gentoo.org/658470
    Closes: https://bugs.gentoo.org/674772
    Signed-off-by: Christian Affolter <c.affolter@purplehaze.ch>
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 www-apps/webdavcgi/Manifest                        |   1 +
 .../files/apache-webdavcgi-1.1-example.conf        |   7 +-
 .../webdavcgi-1.1.2-fix-unescaped-braces.patch     |  98 ++++++++++
 www-apps/webdavcgi/webdavcgi-1.1.2.ebuild          | 206 +++++++++++++++++++++
 4 files changed, 309 insertions(+), 3 deletions(-)