Page MenuHomeFreeBSD

Uses/cmake.mk: Replace CMAKE_ENV with CONFIGURE_ENV.
ClosedPublic

Authored by rakuco on Aug 17 2015, 9:18 PM.
Tags
None
Referenced Files
F81966491: D3403.id8024.diff
Tue, Apr 23, 9:10 PM
Unknown Object (File)
Mon, Apr 22, 4:05 AM
Unknown Object (File)
Thu, Apr 18, 8:16 PM
Unknown Object (File)
Tue, Apr 16, 2:43 PM
Unknown Object (File)
Tue, Apr 9, 12:37 PM
Unknown Object (File)
Tue, Apr 9, 12:29 PM
Unknown Object (File)
Thu, Mar 28, 10:09 AM
Unknown Object (File)
Mar 3 2024, 2:50 AM
Subscribers
None

Details

Summary

Instead of defining a variable that is almost always based on CONFIGURE_ENV, just use CONFIGURE_ENV directly. This also matches the behavior of other ports that do not use autotools (so most ports can just worry about CONFIGURE_ENV). Additionally, the fact that we do not use ?= means we do not have problems if another file in Uses/ needs to set CONFIGURE_ENV (with CMAKE_ENV, the order of the arguments to USES would matter).

Test Plan

All mentioned ports have been tested with the change and build fine on 9.3-i386.

Diff Detail

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

Event Timeline

rakuco retitled this revision from to Uses/cmake.mk: Replace CMAKE_ENV with CONFIGURE_ENV..
rakuco updated this object.
rakuco edited the test plan for this revision. (Show Details)
makc edited edge metadata.
makc added inline comments.
math/eigen3/Makefile
17 ↗(On Diff #8015)

LOCALBASE is incorrectly used here instead of PREFIX

This revision is now accepted and ready to land.Aug 17 2015, 10:38 PM
rakuco added inline comments.
math/eigen3/Makefile
17 ↗(On Diff #8015)

Fixed in rP394563.

This revision was automatically updated to reflect the committed changes.