Page MenuHomeFreeBSD

Remove OSVERSION checks that do not make sense any more.
ClosedPublic

Authored by mat on Apr 2 2015, 4:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 4:18 PM
Unknown Object (File)
Sat, Apr 20, 5:13 AM
Unknown Object (File)
Fri, Apr 19, 9:12 AM
Unknown Object (File)
Fri, Apr 19, 8:01 AM
Unknown Object (File)
Sat, Mar 30, 1:03 PM
Unknown Object (File)
Sat, Mar 30, 1:03 PM
Unknown Object (File)
Sat, Mar 30, 1:03 PM
Unknown Object (File)
Sat, Mar 30, 1:03 PM
Subscribers

Details

Summary

What I did was consider every OSVERSION <op> xxxx, if it is always true or false, simplify the boolean equation.

Then simplify compounds, for example (${OSVERSION} >= 900000 && ${OSVERSION} < 900021) is always true, as is (${OSVERSION} > 900002 || ${OSVERSION} < 900000 && ${OSVERSION} > 800107).

Regarding patches, when an EXTRA_PATCHES is no longer needed, I remove it, when it is always needed, I renamed it, in one case, I merged two patches.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

mat retitled this revision from to Remove OSVERSION checks that do not make sense any more..
mat updated this object.
mat edited the test plan for this revision. (Show Details)
mat added a reviewer: portmgr.
swills added a reviewer: swills.
swills added a subscriber: swills.

Macro lgtm:

This revision is now accepted and ready to land.Apr 2 2015, 5:05 PM
mat edited edge metadata.

Update with swills review.

This revision now requires review to proceed.Apr 2 2015, 5:09 PM
mat updated this revision to Diff 4633.

Closed by commit rP383107 (authored by @mat).