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 Skipped

Unit

Tests Skipped

Build Status

Buildable 67182
Build 64065: arc lint + arc unit

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.