Page MenuHomeFreeBSD
Differential D11405

devel/leiningen: Debashify
ClosedPublic
Actions

Authored by tobik on Jun 29 2017, 3:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jul 19, 3:43 PM
Unknown Object (File)
Sat, Jul 19, 2:14 AM
Unknown Object (File)
Fri, Jul 18, 4:22 AM
Unknown Object (File)
Fri, Jul 18, 3:58 AM
Unknown Object (File)
Fri, Jul 18, 1:34 AM
Unknown Object (File)
Fri, Jul 18, 1:05 AM
Unknown Object (File)
Fri, Jul 11, 11:06 PM
Unknown Object (File)
Tue, Jul 8, 11:10 AM
View All Files
Subscribers
None

Details

Reviewers
lme
mat
Commits
rP444653: - Debashify with a little bit of dark magic and use /bin/sh for lein
Summary
- Debashify and use /bin/sh for lein
- Remove pkg-message with outdated advice
- Add LICENSE_FILE

  Approved by:	??? (mentor)
  Differential Revision:	https://reviews.freebsd.org/D?????
Test Plan

poudriere testport 11.0/amd64 ok

Diff Detail

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

Event Timeline

tobik created this revision.Jun 29 2017, 3:26 PM
mat added inline comments.Jun 29 2017, 3:44 PM
devel/leiningen/Makefile
6–8

You should remove all the :source1 related bits, they are also generated by USE_GITHUB. So you end up having the distfile twice, it is the reason you get it twice when you run make makesum.

30

This should be

${JAVAJARDIR}/leiningen.jar
tobik updated this revision to Diff 30210.Jun 29 2017, 3:55 PM
  • Remove :source1 and fix PLIST_FILES
Harbormaster completed remote builds in B10189: Diff 30210.Jun 29 2017, 3:55 PM
tobik updated this revision to Diff 30211.Jun 29 2017, 3:56 PM
  • Revert distinfo change
mat accepted this revision.Jun 29 2017, 4:27 PM
mat added inline comments.
devel/leiningen/Makefile
36–39

This feels a bit like dark magic :-)

This revision is now accepted and ready to land.Jun 29 2017, 4:27 PM
Closed by commit rP444653: - Debashify with a little bit of dark magic and use /bin/sh for lein (authored by tobik). · Explain WhyJun 29 2017, 4:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
devel/
leiningen/
17 lines

Diff 30211

View Options

devel/leiningen/Makefile

Loading...
View Options

devel/leiningen/pkg-message

Loading...