Bug 223371

Summary: devel/p5-Paws: Update to 0.35
Product: Ports & Packages Reporter: pprocacci
Component: Individual Port(s)Assignee: Jochen Neumeister <joneum>
Status: Closed Feedback Timeout    
Severity: Affects Many People CC: joneum, pprocacci, xmj
Priority: --- Keywords: needs-patch
Version: LatestFlags: bugzilla: maintainer-feedback? (kuriyama)
Hardware: Any   
OS: Any   
See Also: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=238805
Attachments:
Description Flags
Patch to upgrade to latest
none
Patch updates to 0.36 none

Description pprocacci 2017-11-01 19:19:50 UTC
Created attachment 187656 [details]
Patch to upgrade to latest

The current version of p5-Paws is quite old.  Please update to latest version.
The attached patch accomplishes this.  The attached patch also include fixes that aren't yet released.  Specifically:

https://github.com/pplu/aws-sdk-perl/issues/201
and
https://github.com/pplu/aws-sdk-perl/issues/201
Comment 2 pprocacci 2017-11-06 19:56:11 UTC
Created attachment 187808 [details]
Patch updates to 0.36

Update to 0.36
Comment 3 Jochen Neumeister freebsd_committer freebsd_triage 2018-05-13 17:58:07 UTC
Maintainer timeout. Take this PR
Comment 4 Jochen Neumeister freebsd_committer freebsd_triage 2018-05-14 16:06:55 UTC
(In reply to pprocacci from comment #2)

can you upload the diff again? 
I think, the attach has an error
Comment 5 pprocacci 2018-05-18 14:26:06 UTC
The patch applied perfectly for me.

My steps:

cd /usr/ports/devel

fetch -o p5-Paws-0.36.xz "https://bugs.freebsd.org/bugzilla/attachment.cgi?id=187808

unxz p5-Paws-0.36.xz

patch < ./p5-Paws.0.36.diff

make -C ./p5-Paws install clean
Comment 6 Johannes Jost Meixner freebsd_committer freebsd_triage 2018-05-18 16:34:15 UTC
I can confirm it works and applies cleanly as an .xz archive.

That said, the preferred way to upload it would be as plain-text patch: Bugzilla has quite good integration with patches, showing what changes before downloading things.

Please consider that for your next contributions! :)
Comment 7 Jochen Neumeister freebsd_committer freebsd_triage 2018-05-30 07:29:31 UTC
This patch doesn't work, see: http://joneumbox.org/data/11-1-AMD64-ports/2018-05-30_09h25m49s/logs/errors/p5-Paws-0.36.log

====> Compressing man pages (compress-man)
===========================================================================
====> Running Q/A tests (stage-qa)
Error: '/bin/bash' is an invalid shebang you need USES=shebangfix for 'bin/xml2yaml.sh'
Warning: devel/p5-Getopt-Long is present in Perl CORE.  Check with `corelist Getopt::Long 0` and `corelist -a Getopt::Long` if the dependency is really needed or if it should be conditionally added depending on PERL_LEVEL
Warning: www/p5-HTTP-Tiny is present in Perl CORE.  Check with `corelist HTTP::Tiny 0` and `corelist -a HTTP::Tiny` if the dependency is really needed or if it should be conditionally added depending on PERL_LEVEL
Notice: You have some Perl modules as dependencies but you do not have devel/p5-Module-CoreList installed, the perlcore QA check gets better results when using it, especially with older Perl versions.
*** Error code 1

Stop.
make: stopped in /usr/ports/devel/p5-Paws
=>> Error: stage-qa failures detected
=>> Cleaning up wrkdir
===>  Cleaning for p5-Paws-0.36
Comment 8 Kubilay Kocak freebsd_committer freebsd_triage 2019-06-26 07:43:07 UTC
This should have been closed feedback timeout, with a comment clarifying that it was being closed due to not receiving an updated patch to address the failure outlined in comment 7.

Incidentally, please include logs as attachments not external URL references as they tend to go missing/stale over time ( this one is now 404).