Agreed. Big job. Patches welcome :)
-jon
On 06/15/2018 07:27 PM, Chase wrote:
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
--
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
