catalyst works fine on my sh arch with this patch. Please apply this patch and add sh to KEYWORDS.
Created attachment 68253 [details, diff] catalyst-sh.patch
Added to CVS HEAD... (catalyst 2)
it should also declare CXXFLAGS in there ...
Should they be the same as CFLAGS or is there anything special for sh for C++?
just set em equal
<wolf31o2-work> rocket: how does CXXFLAGS normally get set in catalyst? does it normally copy CFLAGS? <rocket> let me double check .. it might <wolf31o2-work> ok <rocket> look at the hppa.py is a way to override it <wolf31o2-work> reason I ask is because of bug #105693 where they're asking to have CFLAGS defined... <jeeves> wolf31o2-work: http://bugs.gentoo.org/show_bug.cgi?id=105693 enh, P2, All, matsuu@gentoo.org->catalyst@gentoo.org, ASSIGNED, pending, dev-util/catalyst: please add sh arch <wolf31o2-work> but I'm noticing that most arches do *not* have it defined <rocket> if self.settings.has_key("CXXFLAGS"): <rocket> myf.write('CXXFLAGS="'+self.settings["CXXFLAGS"]+'"\n') <rocket> else: <rocket> myf.write('CXXFLAGS="${CFLAGS}"\n') <rocket> it will use CFLAGS by default <wolf31o2-work> what file is that in? <wolf31o2-work> nevermind <wolf31o2-work> modules/generic_stage_target.py Anyway, if you have CXXFLAGS that are different from CFLAGS, then I'll add them, otherwise catalyst already sets them to be equivalent.
sounds good, thanks
