STR #3378: configure.ac script changes for compiling fltk on Solaris

STR #3378

Application:	FLTK Library
Status:	1 - Closed w/Resolution
Priority:	3 - Moderate, e.g. unable to compile the software
Scope:	2 - Specific to an operating system
Subsystem:	Build Files
Summary:	configure.ac script changes for compiling fltk on Solaris
Version:	1.3.4
Created By:	nrau
Assigned To:	AlbrechtS
Fix Version:	1.4.0
Fix Commit:	71eda122e8649df6f78e8972119c161c75a2056b
Update Notification:	Receive EMails Don't Receive EMails

Trouble Report Files:

#1	AlbrechtS 08:22 Apr 20, 2017

configure.diff
1k

Trouble Report Comments:

#1	nrau 09:17 Apr 19, 2017

To build a .so on Solaris, we needed to apply a patch to configure.ac which is included below. I am not quite sure how contribution of patches work for fltk, so submitting it as a bug with the requested patch here. --- configure.ac 2017-04-17 18:39:45.435860505 +0000 +++ configure.ac 2017-04-17 18:40:39.007084697 +0000 @@ -230,7 +230,7 @@ GLDSONAME="libfltk_gl.so.$FL_DSO_VERSION" IMGDSONAME="libfltk_images.so.$FL_DSO_VERSION" CAIRODSONAME="libfltk_cairo.so.$FL_DSO_VERSION" - DSOCOMMAND="\$(CXX) \$(DSOFLAGS) -h \$@ \$(LDLIBS) -G $DEBUGFLAG -o" + DSOCOMMAND="\$(CXX) \$(DSOFLAGS) -Wl,-soname,\$@ \$(LDLIBS) -shared -fPIC $DEBUGFLAG -o" if test "x$libdir" != "x/usr/lib"; then DSOLINK="-R$libdir" fi

#2	AlbrechtS 08:22 Apr 20, 2017

Thanks for the report and the proposed patch. You can use the 'Post File' link in the STR form to contribute a patch, preferred in unified diff format (as you included it inline). svn diff, git diff, or diff -u are useful to generate such patches. svn diff has the additional advantage that it contains the svn (subversion) revision (otherwise you could mention it in the text). See attached file configure.diff for reference. Can you please tell us a little more about which version of Solaris needs this patch? Do you know since which version this is necessary? I'm asking because the configure part your patch is changing is explicitly for Solaris, so when did it change? I'd like to add some info to the changes file for reference if you have it handy. TIA.

#3	nrau 12:25 Apr 21, 2017

Hi, I am trying this on the recent released versions of Solaris 11. I haven't tried it on older reeleases, and I am not sure if someone attempted it on earlier versions like Solaris 10.

#4	AlbrechtS 13:59 Jan 16, 2023

Fixed in Git repository. Fixed in commit 71eda122e8649df6f78e8972119c161c75a2056b. Note that this is FLTK 1.4.0 (branch master). FLTK 1.3.x will not be updated with this patch.