After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 558253 - build fails under Solaris
build fails under Solaris
Status: RESOLVED FIXED
Product: libgsf
Classification: Core
Component: General
1.14.x
Other Solaris
: Normal normal
: ---
Assigned To: Morten Welinder
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2008-10-28 15:23 UTC by Jörn Clausen
Modified: 2008-11-05 20:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Tentative patch (1.92 KB, patch)
2008-10-30 18:38 UTC, Morten Welinder
none Details | Review

Description Jörn Clausen 2008-10-28 15:23:25 UTC
Build of 1.14.10 under Solaris 10 fails with

gsf-output-stdio.c: In function `rename_wrapper':
gsf-output-stdio.c:81: error: too few arguments to function `statfs'
gsf-output-stdio.c:82: error: structure has no member named `f_type'

statfs is defined as

int statfs(const char *, struct statfs *, int, int);

in /usr/include/sys/statfs.h.

configure seems to test for the presence of statfs, but not if it is usable (in this way).
Comment 1 Morten Welinder 2008-10-29 19:20:59 UTC
Ok, that's bad but not quite a blocker for most of our users.

It would be helpful if you could come up with a patch that just disables
the code in question if stat is not of the two-arg form we expect.
I don't want a direct test for Solaris as other systems might have
similar issues.
Comment 2 Jörn Clausen 2008-10-30 08:13:38 UTC
The configure script of gnome-vfs defines such a test. The result is used in modules/file-method.c.
Comment 3 Morten Welinder 2008-10-30 18:38:01 UTC
Created attachment 121671 [details] [review]
Tentative patch

Please test this patch.
Comment 4 Jörn Clausen 2008-10-31 12:25:23 UTC
Would you at least look at the code in gnome-vfs? There is more than Linux and the rest of the world. Nearly every OS does statfs differently.

Maybe there is another method to test for a FUSE fs??
Comment 5 Morten Welinder 2008-10-31 13:38:54 UTC
I looked.  It's different and even more complete.  It does not matter.

The point is that I am unable to test changes like this.  Someone else will
have to come up with a patch that does something interesting.  Then I can
look that over and verify that it does no damage on Linux.

(I don't even know if FUSE exists for Solaris.
http://opensolaris.org/os/project/fuse/ suggests that one day it might.)
Comment 6 Morten Welinder 2008-11-05 20:44:29 UTC
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.

We're open for further patches, if needed.