Page MenuHomeFreeBSD

New port: misc/liblxi: Library for communicating with LXI compatible instruments
AbandonedPublic

Authored by yuri on Dec 29 2017, 11:53 PM.
Tags
None
Referenced Files
F81597941: D13685.diff
Thu, Apr 18, 5:46 PM
Unknown Object (File)
Dec 23 2023, 4:49 AM
Unknown Object (File)
Dec 14 2023, 1:44 PM
Unknown Object (File)
Oct 29 2023, 7:45 AM
Unknown Object (File)
Jul 9 2023, 9:55 PM
Unknown Object (File)
Jun 27 2023, 11:36 PM
Unknown Object (File)
Jun 15 2023, 6:49 PM
Unknown Object (File)
Jun 3 2023, 1:39 AM
Subscribers

Diff Detail

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

Event Timeline

misc/liblxi/Makefile
30

^ twice avahi-app

I would remove the static option altogether and never install the .a file.

yuri marked an inline comment as done.Dec 30 2017, 9:08 AM

Why the static library hate? This is my second port where committer was asked to remove static libraries and I'd like to understand the rationale behind it. Are we turning into Arch Linux?

In D13685#286254, @dg_syrec.org wrote:

Why the static library hate? This is my second port where committer was asked to remove static libraries and I'd like to understand the rationale behind it. Are we turning into Arch Linux?

pkg knows when shared libraries have changed, and knows which ports will need to be reinstalled. It has (of course) no knowledge of what was statically linked. Ports should always link against shared libraries except for certain vital programs (like shells) that may need to be run in safe mode without shared libraries being mounted or available (and even then, it's a non-default option).

Hi Adam,

You're right of course about ports linkage, but most library ports are also usable outside of just being a dependency of another port. Users may have (as I do in case of misc/libpostal - the other port where we had this discussion) some proprietary software that needs to be statically linked with liblxi. Removing STATIC option (OFF by default) indeed saves a few KB and 3 lines in the svn diff, but now instead of toggling STATIC checkbox, users need to do an exploratory dive into (probably unfamiliar) port sources.

I don't think that removing static libraries from ports is the correct approach because I believe the choice of whether or not to have static libraries should be down to the port user and we should at least provide a way to easily install them if needed.

misc/liblxi/Makefile
28

It feels a bit strange that you do not need any configure args to enable avahi support.

yuri marked an inline comment as done.Dec 31 2017, 9:47 AM
yuri added inline comments.
misc/liblxi/Makefile
28

That's strange indeed. This is because the upstream doesn't have `--disable-avahi```.
I requested it.

yuri marked 2 inline comments as done.Dec 31 2017, 9:47 AM

Upstream added --disable-avahi.

Testbuilds are fine, looks good to me

yuri abandoned this revision.