Summary: | issues with x11 device as dynamic module | ||
---|---|---|---|
Product: | Ghostscript | Reporter: | Hin-Tak Leung <htl10> |
Component: | Build Process | Assignee: | Default assignee <ghostpdl-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | till.kamppeter |
Priority: | P4 | ||
Version: | master | ||
Hardware: | Other | ||
OS: | Linux | ||
Customer: | Word Size: | --- | |
Attachments: | patch addressing those issues mentioned. |
Description
Hin-Tak Leung
2010-07-30 01:00:40 UTC
The patch does not delete the original gdevemap. We should avoid that the same code piece exists more than once in the source code. Imaging someone fixes a bug in it and forgets the second incarnation. Can you also add a patch for the "easiest" gdevxcmp fix? Thanks. Hin-Tak, forget my comment #2, I have seen now that you have already addressed gdevxcmp in the patch. Found the comment about gdevemap in the other bug. I will test the patch now on an actual Linux box and if it works commit it. Fixed in SVN rev 11555. Thanks for the patch. (In reply to comment #1) > The patch does not delete the original gdevemap. We should avoid that the same > code piece exists more than once in the source code. Imaging someone fixes a > bug in it and forgets the second incarnation. It shouldn't be deleted - as it is used by mac classic build and windows build to load xfonts. The mapping has not changed for 10 years and unlike to change, but I agree it is not elegant. I have thought of another alternative of addressing gsparamx and gdevemap being shared with non-unix builds in core. One can have a smaller list of objects for the shared x11 target than the build-in x11 target. e.g. x11_shared=<most of OBJ> x11_=$(x11_shared) $(GLOBJ)gdevemap.$(OBJ) $(GLOBJ)gsparamx.$(OBJ) $(GLOBJ)X11.so : $(x11alt_) $(x11_shared) $(CCLD) $(LDFLAGS) -shared -o $(GLOBJ)X11.so $(x11alt_) $(x11_shared) \ -L/usr/X11R6/lib -lXt -lSM -lICE -lXext -lX11 $(XLIBDIRS) It is marginally better in that it does't involve duplicating the body of gdevemap, but make the makefile a bit more complicated. |