Bug 216379 - sysutils/parallel: Update to 20170122
Summary: sysutils/parallel: Update to 20170122
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Po-Chuan Hsieh
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-22 17:50 UTC by Danilo G. Baio
Modified: 2017-01-23 19:00 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (howeyc)


Attachments
parallel-20170122.patch (3.46 KB, patch)
2017-01-22 17:50 UTC, Danilo G. Baio
no flags Details | Diff
parallel-20170122-v02.patch (2.55 KB, patch)
2017-01-22 19:39 UTC, Danilo G. Baio
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Danilo G. Baio freebsd_committer freebsd_triage 2017-01-22 17:50:17 UTC
Created attachment 179222 [details]
parallel-20170122.patch

- Update to 20170122
- Create pkg-plist to simplify Makefile

Changelog: http://git.savannah.gnu.org/cgit/parallel.git/plain/NEWS

[Q/A]

portlint: OK (looks fine.)

testport:
   poudriere: i386|amd64 - 10.3 p16     (OK)
   poudriere: i386|amd64 - 11   p7      (OK)
   poudriere: i386|amd64 - 12   r312317 (OK)
Comment 1 howeyc 2017-01-22 18:42:09 UTC
Looks good to me.
Comment 2 Po-Chuan Hsieh freebsd_committer freebsd_triage 2017-01-22 18:58:32 UTC
I would suggest using PORTDOCS=* instead of a long pkg-plist.

BTW, sysutils/moreutils-parallel was removed in Sep 20, 2014. I think it's OK to remove CONFLICTS.
Comment 3 Danilo G. Baio freebsd_committer freebsd_triage 2017-01-22 19:39:59 UTC
Created attachment 179225 [details]
parallel-20170122-v02.patch

Thank you both, for the quick response.
Patch updated as suggested on comment 2.
Another round of poudriere buildtests were fine.
Comment 4 Po-Chuan Hsieh freebsd_committer freebsd_triage 2017-01-22 19:42:31 UTC
And here's the stage-qa result:

====> Running Q/A tests (stage-qa)
Error: '/usr/bin/perl' is an invalid shebang you need USES=shebangfix for 'bin/parcat'
Error: '/bin/bash' is an invalid shebang you need USES=shebangfix for 'bin/env_parallel'
*** Error code 1

Stop.
Comment 5 Po-Chuan Hsieh freebsd_committer freebsd_triage 2017-01-22 19:46:42 UTC
(In reply to Danilo G. Baio (dbaio) from comment #3)

It's OK now. Comment #4 is about the previous patch.
Comment 6 commit-hook freebsd_committer freebsd_triage 2017-01-23 18:53:34 UTC
A commit references this bug:

Author: sunpoet
Date: Mon Jan 23 18:51:57 UTC 2017
New revision: 432281
URL: https://svnweb.freebsd.org/changeset/ports/432281

Log:
  Update to 20170122

  - Remove CONFLICTS: moreutils-parallel was removed in Sep 20, 2014
  - Use PLIST instead of PLIST_FILES
  - While I'm here:
    - Add missing DOCS_BUILD_DEPENDS
    - Remove redundant LOCALBASE/bin in DOCS_BUILD_DEPENDS

  Changes:	http://git.savannah.gnu.org/cgit/parallel.git/plain/NEWS
  PR:		216379
  Submitted by:	Danilo G. Baio (dbaio) <dbaio@bsd.com.br>
  Approved by:	Chris Howey <howeyc@gmail.com> (maintainer)

Changes:
  head/sysutils/parallel/Makefile
  head/sysutils/parallel/distinfo
  head/sysutils/parallel/pkg-plist
Comment 7 Po-Chuan Hsieh freebsd_committer freebsd_triage 2017-01-23 19:00:11 UTC
Committed. Thanks!