Page MenuHomeFreeBSD

Use keyword to simplify ldconfig code
ClosedPublic

Authored by bapt on Jul 15 2019, 8:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 12:48 AM
Unknown Object (File)
Wed, Apr 24, 9:21 PM
Unknown Object (File)
Wed, Apr 24, 9:20 PM
Unknown Object (File)
Wed, Apr 24, 9:20 PM
Unknown Object (File)
Wed, Apr 24, 9:20 PM
Unknown Object (File)
Wed, Apr 24, 9:20 PM
Unknown Object (File)
Wed, Apr 24, 9:20 PM
Unknown Object (File)
Wed, Apr 24, 9:20 PM
Subscribers

Details

Diff Detail

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

Event Timeline

It allows INSTALL_AS_USER to work at the package level

Keywords/ldconfig.ucl
9
if [ -z "${INSTALL_AS_USER}" ]
10

Are there other exit codes for ldconfig ?

bapt added inline comments.
Keywords/ldconfig.ucl
10

Not that I am aware of

This revision is now accepted and ready to land.Jul 15 2019, 12:46 PM

Properly test empty variable

This revision now requires review to proceed.Jul 15 2019, 12:49 PM
Keywords/ldconfig-linux.ucl
9–10

Could this be written as:

%@/sbin/ldconfig

?

Avoid intermediary variable

bapt marked an inline comment as done.Jul 15 2019, 1:01 PM
bapt added inline comments.
Keywords/ldconfig-linux.ucl
9–10

yes done

Keywords/ldconfig-linux.ucl
8

Use $() instead of ``

bapt marked an inline comment as done.

Address latest @mat concerned

bapt marked an inline comment as done.Jul 15 2019, 1:20 PM
This revision is now accepted and ready to land.Jul 15 2019, 2:12 PM
This revision now requires review to proceed.Jul 15 2019, 2:13 PM

has this been committed ? or forgotten ?

I am still waiting for a review ;)

This revision was not accepted when it landed; it landed in state Needs Review.Aug 23 2019, 12:46 PM
This revision was automatically updated to reflect the committed changes.