We really ought to stop patching our version of Imake and just start using the ones prepackaged in Linux, *BSD and Solaris, would make all our lives much easier and the code smaller and more maintainable.
Thank you for your time, -Chase ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On June 15, 2018 5:38 PM, Jon Trulson <j...@radscan.com> wrote: > > > They are all in master, I've deleted cde-next for now. Since I can test > > with fbsd and obsd now, the patches I had in cde-next were moved into > > master yesterday (I think there were only two). > > -jon > > On 06/15/2018 04:36 PM, Matthew R. Trower wrote: > > > Are those fixes in master, or cde-next? > > > > One is Sun-specific (long ago, Ulrich disabled some code by bumping > > compiler version checks; GCC 6 causes that code to run again). The real > > meat though is standards-compliance issues. I don't recall the exact > > error messages, but I've seen them at least on sun when trying to compile > > older codebases with newer GCC. Might just need a flag to tell it which > > standard applies... > > > > I'd slated this work for post release (as we at least now have in-repo > > sunstudio and in-repo GCC 4.x working), but it's possible this problem > > isn't a hard one. I'll investigate a little later and send a build log. > > > > -mrt > > > > Original Message > > > > From: Jon Trulson > > > > Sent: Friday, June 15, 2018 17:29 > > > > To: cdesktopenv-devel@lists.sourceforge.net > > > > Subject: Re: [cdesktopenv-devel] imake gcc version detection (sun) > > > > merged to master, thanks! > > > > What gcc6 issues did you have? I fixed some for FreeBSD, maybe it works > > > > for you now? > > > > -jon > > > > On 06/15/2018 01:31 AM, Matthew R. Trower wrote: > > > > > Hey, I got a clean build out of GCC =) > > > > > > CDE on Sun, when built with GCC, uses whatever GCC binary is in the > > > > > > path. This is good and proper in my opinion. However, imake has all of > > > > > > the compiler paths hardcoded. This causes the version and include paths > > > > > > to be detected improperly on modern OI Hipster, resulting in build > > > > > > failure. The detection code on Sun is also brittle (remember that imake > > > > > > segfault patch?), and quite duplicative. > > > > > > This patch refactors the sun detection code, improves the robustness of > > > > > > said code, and aligns version detection with the actual build process. > > > > > > OI users can now build using GCC 4.x by putting it first in their path > > > > > > (GCC 6.x is still broken). > > > > > > All of the above applies only to Sun. There is probably opportunity to > > > > > > improve this type of code on other systems, but I'm not touching it > > > > > > before release. > > > > > > With this patch (and the others pending approval), I'm almost done; I > > > > > > have two more (minor) patches planned and then I'll be set for release. > > > > > > -- Matthew R. Trower > > > > > > Check out the vibrant tech community on one of the world's most > > > > > > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > > > > > > cdesktopenv-devel mailing list > > > > > > cdesktopenv-devel@lists.sourceforge.net > > > > > > https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel > > -- > > Jon Trulson > > "Fire all weapons and open a hailing frequency for my victory yodle." > > - Zapp Brannigan > > > -------------------------------------------------------------------------------------------------------- > > Check out the vibrant tech community on one of the world's most > > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > > cdesktopenv-devel mailing list > > cdesktopenv-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ cdesktopenv-devel mailing list cdesktopenv-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel
