Page MenuHomeFreeBSD

textproc/git-chglog: Update to 0.13.0
ClosedPublic

Authored by lcook on Mar 25 2021, 12:59 PM.
Tags
None
Referenced Files
F170816209: D29421.id86302.diff
Sun, Sep 6, 7:46 PM
F170802349: D29421.id86302.diff
Sun, Sep 6, 5:46 PM
F170721961: D29421.id.diff
Sun, Sep 6, 5:30 AM
Unknown Object (File)
Fri, Sep 4, 1:52 PM
Unknown Object (File)
Fri, Sep 4, 7:50 AM
Unknown Object (File)
Wed, Sep 2, 6:32 AM
Unknown Object (File)
Mon, Aug 31, 9:44 AM
Unknown Object (File)
Mon, Aug 31, 9:44 AM
Subscribers

Details

Summary
Test Plan

QA:

  • portlint: OK (looks fine).
  • testport: OK (poudriere: 12.2-RELEASE-p4, amd64/i386).

MFH: No (Feature update only).

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 38065
Build 34954: arc lint + arc unit

Event Timeline

LGTM.

Just out of curiosity, why do we need BUILDFLAGS now?

This revision is now accepted and ready to land.Mar 25 2021, 4:16 PM

LGTM.

Just out of curiosity, why do we need BUILDFLAGS now?

Versions before this update would hard-code its version string inside the source. So instead, we can pass the appropriate version during the build process via ldflags. See here for the pull-request. It has become a convention nowadays to do this while also making it easier on the maintainer. In some cases, the author of an application forgets to bump the hard-coded value, so this avoids that entirely when packaging up for a system.

This revision was automatically updated to reflect the committed changes.

LGTM.

Just out of curiosity, why do we need BUILDFLAGS now?

Versions before this update would hard-code its version string inside the source. So instead, we can pass the appropriate version during the build process via ldflags. See here for the pull-request. It has become a convention nowadays to do this while also making it easier on the maintainer. In some cases, the author of an application forgets to bump the hard-coded value, so this avoids that entirely when packaging up for a system.

Thanks for the explanation!