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 117899 - Patch to make it compile on Solaris
Patch to make it compile on Solaris
Status: RESOLVED FIXED
Product: nautilus-cd-burner
Classification: Deprecated
Component: burn vfs-method
unspecified
Other Solaris
: Normal normal
: ---
Assigned To: Nautilus CD Burner Maintainers
Nautilus CD Burner Maintainers
: 105299 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2003-07-20 14:58 UTC by Fredrik Jönsson
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch to nautilus-cd-burner to compile on Solaris (7.13 KB, patch)
2003-07-20 15:00 UTC, Fredrik Jönsson 		none Details | Review
mkdtemp.c (4.88 KB, patch)
2003-07-20 15:01 UTC, Fredrik Jönsson 		none Details | Review
mkdtemp.h (1.38 KB, patch)
2003-07-20 15:02 UTC, Fredrik Jönsson 		none Details | Review

Description Fredrik Jönsson 2003-07-20 14:58:48 UTC 
Below is a patch to make nautilus-cd-burner compile on Solaris. Solaris
lacks both mkdtemp() and statfs(). Instead statvfs() is used and a stolen
mkdtemp() from gettext which includes it for the same reason.

The conditional compilation of mkdtemp.c in Makefile.am should be possible
to do in a nicer way. I know it is with more recsent automake/autoconf.

I'll attach the files for mkdtemp() seperately.
Comment 1 Fredrik Jönsson 2003-07-20 15:00:33 UTC 
Created attachment 18456 [details] [review]
Patch to nautilus-cd-burner to compile on Solaris
Comment 2 Fredrik Jönsson 2003-07-20 15:01:31 UTC 
Created attachment 18457 [details] [review]
mkdtemp.c
Comment 3 Fredrik Jönsson 2003-07-20 15:02:14 UTC 
Created attachment 18458 [details] [review]
mkdtemp.h
Comment 4 Bastien Nocera 2003-07-22 23:22:39 UTC 
*** Bug 105299 has been marked as a duplicate of this bug. ***
Comment 5 Bastien Nocera 2003-07-22 23:29:01 UTC 
I had to make changes to Makefile.am though, it was kinda broken (ie.
mkdtemp.h wouldn't have been distributed, and it made the code in
there more complicated than it should have been)

2003-07-23  Bastien Nocera  <hadess@hadess.net>
                                                                     
          
        * Makefile.am:
        * configure.in:
        * make-iso.c: (make_iso):
        * mapping-daemon.c: (main):
        * mapping-method.c: (vfs_module_init):
        * mkdtemp.c:
        * mkdtemp.h: patch from Fredrik Jönsson <fjo@nada.kth.se> to
provide
        statfs replacement on Solaris, and mkdtemp.[ch] on non-OpenBSD,
        or non-glibc systems, use the same code in mapping-method.c as in
        mapping-daemon.c to create the communication socket (Closes:
        #117948, #117899)