Page MenuHomeFreeBSD

comms/tio: Update to 1.26
ClosedPublic

Authored by yuri on Nov 8 2017, 12:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 21 2024, 7:14 AM
Unknown Object (File)
Feb 21 2024, 7:14 AM
Unknown Object (File)
Feb 21 2024, 7:14 AM
Unknown Object (File)
Feb 21 2024, 7:01 AM
Unknown Object (File)
Jan 22 2024, 7:15 PM
Unknown Object (File)
Dec 22 2023, 11:27 PM
Unknown Object (File)
Dec 21 2023, 8:16 PM
Unknown Object (File)
Nov 18 2023, 5:14 PM
Subscribers

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tcberner added inline comments.
comms/tio/Makefile
26 ↗(On Diff #34907)

^ I sometimes wonder wheter a single file is really worth an OPTION ^^ -- not really related ^^

This revision is now accepted and ready to land.Nov 8 2017, 6:32 AM
yuri marked an inline comment as done.Nov 8 2017, 6:35 AM
yuri added inline comments.
comms/tio/Makefile
26 ↗(On Diff #34907)

Some people want to install all BASH bindings, so that they can just set BASH=yes for everything. It's nice IMO, even though it's just one file. :)

This revision was automatically updated to reflect the committed changes.
yuri marked an inline comment as done.
comms/tio/Makefile
26 ↗(On Diff #34907)

Yes, but those who don't hopeuflly have other things to worry about than a 2.2k text file more on there disk ^^ -- what I was suggesting is to always just install such things :)

Yes, I know, but where do you draw the line? Having an option is certainly very beneficial when it enables/disables large parts. Some more pedantically-inclined people with argue that it's warranted even for such small single files. Packages in Gentoo linux, I think, do this a lot. There is likely no consensus on this in the computer community. :)

head/comms/tio/Makefile
26–27

Never, ever, put %%FOO%% in PLIST_FILES. The idea behind %%FOO%% placeholders is to allow one to carry variables from the Makefile to the pkg-plist file. Here, you are in the Makefile, simply use the variables, or in that case, do it the way the framework helps you to do it:

PLIST_FILES=        bin/tio \
	            man/man1/tio.1.gz
BASH_PLIST_FILES=   share/bash-completion/completions/tio
PORTDOCS            README

Also, whenever you install files in DOCSDIR, you MUST add a DOCS option.

I am STRONGLY in favour of more options. Even if it's a 1-file change. Give people choice and control. This is BSD, after all.

I created the request for portlint to complain about %%FOO%% in PLIST_FILES: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=223539

I've asked portlint to require DOCS when %%PORTDOCS%% is used in plist: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=223541

So do I have an approval to change these two items:

  • Restore pkg-plist and remove PKG_PLIST from Makefile
  • Add DOCS to options

?

yuri marked an inline comment as done.Nov 8 2017, 6:08 PM