devel/git - Lighten perl dependencies
ClosedPublic
Actions

Authored by adamw on Feb 3 2016, 9:55 PM.

Details

Reviewers

Commits

rP408063: Lighten up the Perl dependencies.

Summary

See bug#206901. The perl hooks only require p5-Error, and the other two modules (which bring in a half dozen other modules) are only used for git-send-email(1).

This patch adds a SEND_EMAIL option (which auto-enables the PERL option) and conditionalizes the git-send-email script installation.

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

adamw updated this revision to Diff 12983.Feb 3 2016, 9:55 PM

adamw retitled this revision from to devel/git - Lighten perl dependencies.

adamw updated this object.

adamw edited the test plan for this revision. (Show Details)

adamw added a reviewer: garga.

adamw set the repository for this revision to rP FreeBSD ports repository.

It looks good to me. If it passes all tests with poudriere go ahead and commit it.

This revision is now accepted and ready to land.Feb 4 2016, 9:59 AM

It changes the default package because the git-send-email(1) script (and the other perl modules) were installed by default before. Should SEND_EMAIL default to off (changes default package), or on (default package stays the same)?

Please leave the option enabled by default since it's a common used feature

Closed by commit rP408063: Lighten up the Perl dependencies. (authored by adamw). · Explain WhyFeb 4 2016, 5:33 PM

This revision was automatically updated to reflect the committed changes.

Done. Thanks, Renato!