devel/gnulib: Fix shebangfix for clisp
ClosedPublic
Actions

Authored by ivy on Fri, Sep 19, 8:58 PM.

Details

Reviewers

fuz
arrowd

Commits

R11:ca139ab695b7: devel/gnulib: Fix shebangfix for clisp

Summary

Uses/shebangfix.mk doesn't know about clisp, and without this, stage-qa
complains when ${LOCALBASE} is not /usr/local since we don't replace the
shebang.

Diff Detail

Repository

R11 FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ivy requested review of this revision.Fri, Sep 19, 8:58 PM

ivy created this revision.

Harbormaster completed remote builds in B67182: Diff 162441.Fri, Sep 19, 8:58 PM

LGTM, but requires a PORTREVISION bump.

This revision is now accepted and ready to land.Sat, Sep 20, 5:14 AM

In D52630#1202358, @arrowd wrote:

LGTM, but requires a PORTREVISION bump.

i'll do that, but could you briefly explain why? for me this is a build fix, so if the previous version of the port built, then the shebang path is already correct and the update won't change anything. i assume this is different for people who don't have DEVELOPER set?

Closed by commit R11:ca139ab695b7: devel/gnulib: Fix shebangfix for clisp (authored by ivy). · Explain WhySat, Sep 20, 3:48 PM

This revision was automatically updated to reflect the committed changes.

ivy added a commit: R11:ca139ab695b7: devel/gnulib: Fix shebangfix for clisp.

Yes, you're right, the bump wasn't really needed. I misread the change and was thinking that the current package has a wrong shebang.