On Tue, Nov 08, 2016 at 09:56:36PM +0100, Adam Wolk wrote:
> On Tue, Nov 08, 2016 at 09:43:45PM +0100, Joerg Jung wrote:
> > 
> > > Am 08.11.2016 um 15:11 schrieb Adam Wolk <adam.w...@tintagel.pl>:
> > > 
> > > Hi ports@,
> > > 
> > > The cloc project moved from source forge to github and
> > > had a new release v1.70 (July 1, 2016).
> > > 
> > > I'm attaching a diff moving to the new MASTER_SITE
> > > and bumping version to v1.70.
> > > 
> > > CC'ing MAINTAINER.
> > > 
> > > Looking for feedback, OK's.
> > 
> > Can we please switch to use GH_* variables instead of hardcoding 
> > GitHub path? Also please consider switching to the source tarballs,
> > adding the necessary dependencies and install the man page, e.g. read: 
> > https://github.com/AlDanial/cloc/blob/master/Unix/why_this_directory_exists.txt
> > 
> 
> According to Makefile.template:
> # For github: traditional tarballs in /releases/download are normally
> # preferred if available; use them with DISTNAME/MASTER_SITES lines as
> # normal
> 
> using GH_* variables would default to auto generated tarballs that could
> change.
> 
> I will take a look at packaging the tarball with deps & manpage.
> 

I'm attaching a new tarball that is built against the latest 1.70 release.

We had to patch the Makefile hence the tarball vs a diff since I don't want
to force a cvs add for the patches folder before a diff review.

I did add an upstream pull request for part of the Makefile changes:
 - https://github.com/AlDanial/cloc/pull/136

but the second part (/usr/local vs /usr) felt out of scope for a pull request.

Significant changes:
 - do-install is gone
 - LIB_DEPENDS on 2 existing perl 5 ports
 - changing WRKBUILD for the UNIX directory
 - adding 2 patches for Unix/Makefile, one upstreamed (see above)
 - updating PLIST to grab the manpage

Regards,
Adam

Attachment: cloc-1.70.tar.gz
Description: application/tar-gz

Reply via email to