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)
Sat, Apr 27, 7:52 AM
Unknown Object (File)
Sat, Apr 20, 1:06 PM
Unknown Object (File)
Dec 22 2023, 11:51 PM
Unknown Object (File)
Nov 20 2023, 10:55 AM
Unknown Object (File)
Nov 16 2023, 6:02 PM
Unknown Object (File)
Nov 16 2023, 6:01 PM
Unknown Object (File)
Nov 16 2023, 6:00 PM
Unknown Object (File)
Nov 11 2023, 9:23 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.