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
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 19124
Build 18749: 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
20–24

Can't all this be replaced with using GH_SUBDIR?

devel/lazygit/Makefile
20–24

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

Why CP and not MV ?

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

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.