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 696515 - Fix build on non-Linux platforms
Fix build on non-Linux platforms
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: power
unspecified
Other All
: Normal normal
: ---
Assigned To: Richard Hughes
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2013-03-24 20:50 UTC by Emilio Pozuelo Monfort
Modified: 2013-03-27 12:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix build on non-Linux platforms (793 bytes, patch)
2013-03-24 20:50 UTC, Emilio Pozuelo Monfort
committed Details | Review

Description Emilio Pozuelo Monfort 2013-03-24 20:50:44 UTC
power: fails to build when gudev disabled (non-linux)

NULL needs stdlib.h. With gudev support enabled we get
that through gudev.h, but when gudev support is disabled
we don't get it. So just include stdlib.h unconditionally.
Comment 1 Emilio Pozuelo Monfort 2013-03-24 20:50:46 UTC
Created attachment 239698 [details] [review]
Fix build on non-Linux platforms

We need stdlib.h to get the definition of NULL.
Comment 2 Matthias Clasen 2013-03-25 10:21:21 UTC
Review of attachment 239698 [details] [review]:

sure
Comment 3 Bastien Nocera 2013-03-25 10:43:16 UTC
Review of attachment 239698 [details] [review]:

After the freeze is better...
Comment 4 Emilio Pozuelo Monfort 2013-03-25 10:52:08 UTC
I have pushed it to master, I'll wait till 3.8 is released to push it to gnome-3-8. Thanks!
Comment 5 Emilio Pozuelo Monfort 2013-03-27 12:00:55 UTC
Pushed to gnome-3-8.