Le 22/11/2012 13:00, Schindler Karl-Michael a écrit :
Dear Maurice The comma after "-install_name" needs to be a blank in macosx.diff:
- $(CC) $(LDOPT) -dynamiclib -install_name,$(libdir)/$(notdir $(GRX20SHna)) -headerpad_max_install_names -o $(GRX20SHna) $(LO) $(ADDON_LIBS) $(X11LIBS)
- $(CC) $(LDOPT) -dynamiclib -install_name $(libdir)/$(notdir $(GRX20SHna)) -headerpad_max_install_names -o $(GRX20SHna) $(LO) $(ADDON_LIBS) $(X11LIBS)
OK corrected
As such the install_name is what most expect and is fine. In case fink reviewers want a change, I will deal with this within the package description.
I also tried the fat options. In general, they work. However, when I did -arch ppc, 2 small glitches showed up:
- src/include/grx20.h, line 112: there is one unmatched ")" at the end of the line:
-#if defined(unix) || defined(__unix) || defined(__unix__) || defined(_AIX) || (defined(__APPLE__) && defined(__MACH__)) ) +#if defined(unix) || defined(__unix) || defined(__unix__) || defined(_AIX) || (defined(__APPLE__) && defined(__MACH__))
OK corrected
- src/fonts/fdv_raw.c: error: fhdr has no member length. I think it is a
leftover from earlier and therefore I simply deleted line 53:
- _GR_swap32u(&fhdr.length);
in fact length has been changed to numchars, to be consistent with other drivers. But nobody has yet used this instruction because previously all systems were little endian. Corrected anaway
-arch ppc64 compiled, but there is an error during link, because of missing libraries. If I recall correctly, it is not supported on 10.6 and i would have to try on 10.5. I am not sure, whether it is worth to put more work into this.
Probably not, as indicated in an other mail by Adriaan. power pc seems to be outdated now. I suppressed the support in makedefs.grx and configure. But I kept FAT_I386 and FAT_X86_64 to enable a building of a fat library in one go.
This is uploaded to "more things"
I will see for the fpc support.
Maurice