Bug 203846 - [patch] devel/apache-rat: Release audit tool (new port)
Summary: [patch] devel/apache-rat: Release audit tool (new port)
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Cy Schubert
URL:
Keywords: feature, patch, patch-ready
Depends on:
Blocks:
 
Reported: 2015-10-17 23:06 UTC by Pedro F. Giffuni
Modified: 2016-08-30 19:54 UTC (History)
3 users (show)

See Also:
koobs: maintainer-feedback+

Attachments
Shar based executable script (3.29 KB, application/x-shellscript)
2015-10-17 23:06 UTC, Pedro F. Giffuni 		no flags Details
Shar based executable script (3.19 KB, text/plain)
2015-10-18 00:01 UTC, Pedro F. Giffuni 		koobs: maintainer-approval+
Details
New devel/apache-rat port as a diff (4.64 KB, patch)
2016-04-02 21:15 UTC, Pedro F. Giffuni 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Pedro F. Giffuni freebsd_committer freebsd_triage 2015-10-17 23:06:15 UTC 
Created attachment 162173 [details]
Shar based executable script

New port: Apache Release Audit Tool

http://creadur.apache.org/rat/

Use Rat to improve accuracy and efficiency when checking releases for licenses.
Comment 1 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-10-18 00:01:38 UTC 
Created attachment 162178 [details]
Shar based executable script

Small port bugfix.
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2015-10-18 03:34:03 UTC 
Pedro, if you'd like to create a review for this in phabric, im sure a few ports committers would be happy to review/approve. Feel free to assign this issue to yourself if you'd like to do that
Comment 3 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-10-18 16:21:57 UTC 
(In reply to Kubilay Kocak from comment #2)
Thank you for the offer but I don't have much time for this ATM.

The port is rather important though: it let's us do license audits
within the project.
Comment 4 Kubilay Kocak freebsd_committer freebsd_triage 2015-10-19 03:56:51 UTC 
@Pedro, does this pass portlint -AC and poudriere?
Comment 5 Pedro F. Giffuni freebsd_committer freebsd_triage 2015-10-19 16:08:14 UTC 
(In reply to Kubilay Kocak from comment #4)
/usr/ports/devel/apache-rat % portlint -AC
WARN: Makefile: using hyphen in PORTNAME. consider using PKGNAMEPREFIX and/or PKGNAMESUFFIX.
WARN: Consider to set DEVELOPER=yes in /etc/make.conf
0 fatal errors and 2 warnings found.
______
For the warning hyphen, it's the same for apache-ant and apache-poi.

This doesn't have options so poudriere should have no effect.
It passes check-plist.
Comment 6 Martin Wilke freebsd_committer freebsd_triage 2015-12-05 09:35:21 UTC 
Take.
Comment 7 Martin Wilke freebsd_committer freebsd_triage 2016-03-14 02:22:41 UTC 
back to pool.
Comment 8 Pedro F. Giffuni freebsd_committer freebsd_triage 2016-04-02 21:15:32 UTC 
Created attachment 168910 [details]
New devel/apache-rat port as a diff

I generally prefer "svn diff" over shar.

Make the new port available as a patch, given the shar version hasn't received much attention.
Comment 9 commit-hook freebsd_committer freebsd_triage 2016-08-30 19:49:58 UTC 
A commit references this bug:

Author: cy
Date: Tue Aug 30 19:49:27 UTC 2016
New revision: 421130
URL: https://svnweb.freebsd.org/changeset/ports/421130

Log:
  Introduce the new apache-rat 0.11 port.

  Apache Rat is a release audit tool, focused on licenses. Use Rat to improve
  accuracy and efficiency when checking releases for licenses.

  WWW: http://creadur.apache.org/rat/

  PR:		203846
  Submitted by:	pfg@

Changes:
  head/devel/apache-rat/
  head/devel/apache-rat/Makefile
  head/devel/apache-rat/distinfo
  head/devel/apache-rat/files/
  head/devel/apache-rat/files/apache-rat.sh.in
  head/devel/apache-rat/pkg-descr
  head/devel/apache-rat/pkg-plist
Comment 10 Cy Schubert freebsd_committer freebsd_triage 2016-08-30 19:54:26 UTC 
Committed. Thanks for the port.