Page MenuHomeFreeBSD

New port: devel/lazygit: Simple terminal UI for git commands
ClosedPublic

Authored by meta on Aug 23 2018, 7:55 AM.
Tags
None
Referenced Files
F105002992: D16862.id47367.diff
Wed, Dec 11, 1:15 PM
F105002348: D16862.id47155.diff
Wed, Dec 11, 1:05 PM
Unknown Object (File)
Fri, Dec 6, 5:26 PM
Unknown Object (File)
Tue, Dec 3, 8:51 AM
Unknown Object (File)
Tue, Dec 3, 7:23 AM
Unknown Object (File)
Tue, Dec 3, 12:44 AM
Unknown Object (File)
Mon, Nov 25, 12:22 PM
Unknown Object (File)
Sun, Nov 24, 1:06 AM
Subscribers

Details

Summary

I'm not 100% sure how to create a golang port. It fails to find out packages
in the repository. I copy some directories in the repository to the library
search path in post-extract target, it worked.

Test Plan
  • portlint -AC looks fine
  • poudriere test build finished successfully
  • confirmed application actually works fine

Diff Detail

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

Event Timeline

Oops, Arcanist includes the whole diff in the repository in contrast svn only includes the current directory by default. Removed, irrerevant changes.

devel/lazygit/Makefile
5

For new ports, set DISTVERSION instead of PORTVERSION.

devel/lazygit/Makefile
5

Could you show me a pointer? Maybe somewhere in porter's handbook but I cannot find it.

use DISTVERSION rather than PORTVERSION

devel/lazygit/Makefile
5

understood, thanks!

linimon retitled this revision from New port: devel/lagygit: Simple terminal UI for git commands to New port: devel/lazygit: Simple terminal UI for git commands.Aug 24 2018, 4:39 AM
devel/lazygit/Makefile
21โ€“25

Can't all this be replaced with using GH_SUBDIR?

devel/lazygit/Makefile
21โ€“25

I think this is not the case for GH_SUBDIR. No additional distribution files needed. Everything's included in the distribution but the path is wrong. So I copy them to the path where go searches during the build.

Will you update the new port to 0.1.80 before you commit ? Testbuilds look fine.

devel/lazygit/Makefile
25

Why CP and not MV ?

This revision is now accepted and ready to land.Aug 27 2018, 8:10 PM
devel/lazygit/Makefile
25

Whichever is OK to me.

This revision now requires review to proceed.Aug 28 2018, 12:21 AM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 28 2018, 12:30 AM
This revision was automatically updated to reflect the committed changes.