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
Unknown Object (File)
Fri, Nov 8, 3:56 AM
Unknown Object (File)
Wed, Nov 6, 9:56 AM
Unknown Object (File)
Tue, Nov 5, 6:41 AM
Unknown Object (File)
Sun, Oct 27, 6:27 AM
Unknown Object (File)
Thu, Oct 17, 10:37 AM
Unknown Object (File)
Thu, Oct 17, 4:46 AM
Unknown Object (File)
Wed, Oct 16, 12:31 PM
Unknown Object (File)
Wed, Oct 16, 4:57 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
Lint Not Applicable
Unit
Tests Not Applicable

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
4 ↗(On Diff #47156)

For new ports, set DISTVERSION instead of PORTVERSION.

devel/lazygit/Makefile
4 ↗(On Diff #47156)

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
4 ↗(On Diff #47156)

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
20–24 ↗(On Diff #47219)

Can't all this be replaced with using GH_SUBDIR?

devel/lazygit/Makefile
20–24 ↗(On Diff #47219)

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
24 ↗(On Diff #47219)

Why CP and not MV ?

This revision is now accepted and ready to land.Aug 27 2018, 8:10 PM
devel/lazygit/Makefile
24 ↗(On Diff #47219)

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.