Created attachment 173102 [details]
patch to update math/ocamlgsl
The patch updates math/ocamlgsl from 1.15.1 to 1.19.1.
Without this update, math/ocamlgsl would not build, once math/gsl will be updated to version 2.1 (math/gsl update pending, see bug #210168).
- Sources change from http://bitbucket.org/mmottl/gsl-ocaml/downloads/
to github. The github solution here seems not optimal, but it works
- updated patch files/setup.ml
- updated pkg-plist
Tested with Poudriere (HEAD a/i, 10.3 a 9.3 a/i).
'portlint -AC' complains about DOCSDIR in pkg-plist. This is like in version 1.15.1, also contructed by 'make makeplist' ...
thank you for your patch! I (maintainer) am currently on holiday break
and will not be able to test it. But reviewing it raised the following
point, which relates to the warning you mention:
> 'portlint -AC' complains about DOCSDIR in pkg-plist. This is like in version 1.15.1, also contructed by 'make makeplist' ...
Look at the documentation files in pkg-plist. In the old version, they are
prefixed with %%PORTDOCS%% but not in your new version. Fixing this should
remove the warning generated by portlint.
Side question: what are your concerns about using GitHub as a source?
It is “just as bad” as bitbucket. :)
@committer : Reviewing the patch raised a minor issue in the change,
being on holiday break I cannot test for 2 weeks… If you cannot test,
maybe Rainer could add poudriere logs to the ticket and you could skim
on it? But if waiting is also an option, I'll take care of the details
when I'm back.
Created attachment 173113 [details]
revised patch to update math/ocamlgsl
thank you for breaking the break and having a look into the PR ;)
%%PORTDOCS%% indeed is necessary, to respect enabled/disabled option DOCS.
%%DOCSDIR%% is the one, portlint complains about, and which should be used to have a changeable path to the docs. With the dilemma of divergent PORTNAME and DISTNAME, I tried a somewhat ugly hack and redefined DOCSDIR within Makefile. At least this builds and installs fine, with and without option DOCS enabled.
I had to use GitHub, because the old repository has been deleted. Probably someone will find a more elegant way to define the GitHub source.
The newer patch handles DOCSDIR and PORTSDIR in Makefile and pkg-plist in a better way, so that 'portlint -AC' does not complain any more.
testbuilds are fine
A commit references this bug:
Date: Fri Oct 21 14:35:12 UTC 2016
New revision: 424419
math/ocamlgsl: update 1.15.1 -> 1.19.1
- master site changed from bitbucket to github
Submitted by: Rainer Hurling <email@example.com>
Approved by: Michael Gruenewald <firstname.lastname@example.org> (maintainer timeout)