I don't remember seeing anything about this in the changelogs for setuptools.
What ports did you see the problem with? And what python version(s)? I remember in the past some setup.pys with a lot of custom code had similar issues, but ones just using the defaults (which is most of them) didn't. If it is a general problem, it could also have been introduced through a change in distutils. If that's the case, older pythons would be unaffected. - Josh On 2015-1-2 10:28 , Jeremy Huddleston Sequoia wrote: > I suspect that this change became necessary as a result of changes in > behavior in py-setuptools. My guess it that previously, the install phase > did not invalidate the build and rebuild if the environment changed, and now > it does. py-setuptools has had frequent updates over the past year. I did > not determine which version introduced this behavioral change. > > Does anyone know what version introduced this change? > > How should we handle revbumps of affected ports? Ideally, we should just > revbump all python ports that install compiled code, but I'm not sure what > that set is. > >> On Jan 1, 2015, at 13:56, [email protected] wrote: >> >> Revision >> 130565 >> Author >> [email protected] >> Date >> 2015-01-01 13:56:35 -0800 (Thu, 01 Jan 2015) >> Log Message >> >> python-1.0: Set environment variables during destroot >> >> If CC and others are set differently between build and destroot, the port >> will be debuilt during the destroot phase due to the change. This can >> result in the port being rebuild incorrectly (eg: for the compiler default >> architecture instead of the chosen arch, or not universal). >> >> Modified Paths >> >> • trunk/dports/_resources/port1.0/group/python-1.0.tcl >> Diff >> >> Modified: trunk/dports/_resources/port1.0/group/python-1.0.tcl (130564 => >> 130565) >> >> --- trunk/dports/_resources/port1.0/group/python-1.0.tcl 2015-01-01 >> 16:20:54 UTC (rev 130564) >> +++ trunk/dports/_resources/port1.0/group/python-1.0.tcl 2015-01-01 >> 21:56:35 UTC (rev 130565) >> >> @@ -115,24 +115,36 @@ >> >> pre-build { >> >> if {${python.add_archflags}} { >> >> if {[variant_exists universal] && [variant_isset >> universal]} { >> >> - build.env-append CFLAGS="${configure.universal_cflags}" >> \ >> - >> OBJCFLAGS="${configure.universal_cflags}" \ >> - >> CXXFLAGS="${configure.universal_cxxflags}" \ >> - >> LDFLAGS="${configure.universal_ldflags}" >> >> + build.env-append >> CFLAGS="${configure.universal_cflags}" \ >> + >> OBJCFLAGS="${configure.universal_cflags}" \ >> + >> CXXFLAGS="${configure.universal_cxxflags}" \ >> + >> LDFLAGS="${configure.universal_ldflags}" >> + destroot.env-append >> CFLAGS="${configure.universal_cflags}" \ >> + >> OBJCFLAGS="${configure.universal_cflags}" \ >> + >> CXXFLAGS="${configure.universal_cxxflags}" \ >> + >> LDFLAGS="${configure.universal_ldflags}" >> >> } else { >> >> - build.env-append CFLAGS="${configure.cc_archflags}" \ >> - >> OBJCFLAGS="${configure.objc_archflags}" \ >> - CXXFLAGS="${configure.cxx_archflags}" \ >> - FFLAGS="${configure.f77_archflags}" \ >> - F90FLAGS="${configure.f90_archflags}" \ >> - FCFLAGS="${configure.fc_archflags}" \ >> - LDFLAGS="${configure.ld_archflags}" >> >> + build.env-append CFLAGS="${configure.cc_archflags}" \ >> + >> OBJCFLAGS="${configure.objc_archflags}" \ >> + >> CXXFLAGS="${configure.cxx_archflags}" \ >> + FFLAGS="${configure.f77_archflags}" >> \ >> + >> F90FLAGS="${configure.f90_archflags}" \ >> + FCFLAGS="${configure.fc_archflags}" >> \ >> + LDFLAGS="${configure.ld_archflags}" >> + destroot.env-append CFLAGS="${configure.cc_archflags}" \ >> + >> OBJCFLAGS="${configure.objc_archflags}" \ >> + >> CXXFLAGS="${configure.cxx_archflags}" \ >> + FFLAGS="${configure.f77_archflags}" >> \ >> + >> F90FLAGS="${configure.f90_archflags}" \ >> + FCFLAGS="${configure.fc_archflags}" >> \ >> + LDFLAGS="${configure.ld_archflags}" >> >> } >> >> } >> >> if {${python.set_compiler}} { >> >> foreach var {cc objc cxx fc f77 f90} { >> >> if {[set configure.${var}] ne ""} { >> >> build.env-append [string toupper $var]="[set >> configure.${var}]" >> >> + destroot.env-append [string toupper $var]="[set >> configure.${var}]" >> >> } >> >> } >> >> } >> >> _______________________________________________ >> macports-changes mailing list >> [email protected] >> https://lists.macosforge.org/mailman/listinfo/macports-changes > > _______________________________________________ macports-dev mailing list [email protected] https://lists.macosforge.org/mailman/listinfo/macports-dev
