Page MenuHomeFreeBSD

Change default directory used by USES=autoreconf to WRKSRC
ClosedPublic

Authored by jbeich on Nov 15 2015, 7:45 PM.
Tags
None
Referenced Files
F89304838: D4169.id.diff
Fri, Jul 26, 10:06 AM
F89304824: D4169.id10215.diff
Fri, Jul 26, 10:05 AM
F89304822: D4169.id14640.diff
Fri, Jul 26, 10:05 AM
F89304578: D4169.id15145.diff
Fri, Jul 26, 10:02 AM
F89304575: D4169.id10209.diff
Fri, Jul 26, 10:01 AM
F89304574: D4169.id10240.diff
Fri, Jul 26, 10:01 AM
F89304570: D4169.id10216.diff
Fri, Jul 26, 10:01 AM
Unknown Object (File)
Tue, Jul 23, 4:45 PM
Subscribers
None

Details

Test Plan

USES=autoreconf ports that change CONFIGURE_WRKSRC:

  • graphics/p5-ming
  • graphics/py-ming

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jbeich retitled this revision from to Change default directory used by USES=autoreconf to WRKSRC.
jbeich updated this object.
jbeich edited the test plan for this revision. (Show Details)
jbeich added reviewers: portmgr, tijl.
jbeich edited edge metadata.
jbeich edited the test plan for this revision. (Show Details)
  • Fix graphics/p[5y]-ming by not poking USE_GITHUB variables instead of redefining AUTORECONF_WRKSRC
jbeich edited edge metadata.

Oops, use PATCH_WRKSRC in post-patch to avoid confusion.

tijl edited edge metadata.

For me the patch is fine, but I would just use ${WRKSRC}/perl_ext (or py_ext) in post-patch. Since autoreconf runs in WRKSRC it's possible that in the future some files may need to be patched there and then a non-default PATCH_WRKSRC can be a bit confusing or unexpected.

jbeich edited edge metadata.
In D4169#87753, @tijl wrote:

I would just use ${WRKSRC}/perl_ext (or py_ext) in post-patch.

graphics/py-ming already relies on non-default PATCH_WRKSRC as it modifies WRKSRC. I've tried to keep p5-ming consistent with it. However, rebasing the patches would complicate this differential even more.

Let's go back to AUTORECONF_WRKSRC then. Improving p[5y]-ming ports is out of scope here.

This revision was automatically updated to reflect the committed changes.