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 676269 - gnome-session fails to build due to undefined reference
gnome-session fails to build due to undefined reference
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: gnome-session
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Session Maintainers
Session Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-05-17 18:53 UTC by D.S. (Spider) Ljungmark
Modified: 2012-05-18 21:10 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
configure: Check for new enough version of systemd-login (877 bytes, patch)
2012-05-17 20:08 UTC, Colin Walters 		none Details | Review
configure: Check for new enough version of systemd-login (877 bytes, patch)
2012-05-18 18:14 UTC, Colin Walters 		committed Details | Review

Description D.S. (Spider) Ljungmark 2012-05-17 18:53:26 UTC 
gnome_session-gsm-systemd.o: In function `gsm_systemd_is_login_session':
/home/spider/Projects/gnome/gnome-session/gnome-session/gsm-systemd.c:364: undefined reference to `sd_session_get_service'
collect2: ld returned 1 exit status


sd_session_get_service should be defined in <systemd/sd-login.h>

This function was included in upstream systemd-38 and forwards 
( http://cgit.freedesktop.org/systemd/systemd/commit/src/systemd/sd-login.h?id=eff406331adb23e27e4bd29a3b69322fc359ca3d )


The version of systemd should be checked, or the existance of the function that is needed should be checked.

--

Bug occurs on branch master, commit :  ce9e5da40aacfd6dc70dcdb3fc367cff746ded78
Comment 1 Colin Walters 2012-05-17 20:08:27 UTC 
Created attachment 214277 [details] [review]
configure: Check for new enough version of systemd-login

37 as shipped in Fedora 16 isn't.
Comment 2 Ray Strode [halfline] 2012-05-17 20:12:49 UTC 
needs 38, not 43 (see comment 0)
Comment 3 Colin Walters 2012-05-18 18:14:12 UTC 
Created attachment 214392 [details] [review]
configure: Check for new enough version of systemd-login

Now with version 38
Comment 4 Matthias Clasen 2012-05-18 21:08:27 UTC 
Review of attachment 214392 [details] [review]:

Looks good to me
Comment 5 Colin Walters 2012-05-18 21:10:42 UTC 
Attachment 214392 [details] pushed as 12ffa57 - configure: Check for new enough version of systemd-login