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 767368 - unnecessary dependency on uuidgen which isn't checked for by jhbuild.
unnecessary dependency on uuidgen which isn't checked for by jhbuild.
Status: RESOLVED FIXED
Product: gjs
Classification: Bindings
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gjs-maint
gjs-maint
Depends on:
Blocks:
 
 
Reported: 2016-06-07 17:10 UTC by Simon McVittie
Modified: 2016-06-07 18:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
tests: use dbus-uuidgen instead of uuidgen (2.46 KB, patch)
2016-06-07 17:23 UTC, Simon McVittie
committed Details | Review

Description Simon McVittie 2016-06-07 17:10:19 UTC
gjs requires the uuidgen executable, which doesn't seem to be checked for by jhbuild sysdeps. In particular, gjs will fail to build in this setup:

* Debian testing/unstable in a virtual machine
* Install build-essential plus everything requested by `jhbuild sysdeps`
* uuid-dev (development files for libuuid) is present
* uuid-runtime (uuidgen and uuidd) is not present

Looking at how gjs uses uuidgen, it seems dbus-uuidgen would be entirely sufficient - all it wants is an unpredictable string, it doesn't care about the specific form of that string. dbus-uuidgen is part of a package that the D-Bus tests will need anyway, since it's typically packaged alongside dbus-daemon. (That's why I'm filing this as a gjs bug, and not a jhbuild moduleset bug.)

Preparing a patch.
Comment 1 Simon McVittie 2016-06-07 17:23:35 UTC
Created attachment 329317 [details] [review]
tests: use dbus-uuidgen instead of uuidgen

uuidgen is not always installed, even if libuuid is: for example,
in Debian derivatives it's packaged in uuid-runtime alongside uuidd,
a daemon that is in practice nearly always unnecessary.

The only use of uuidgen is in a script that already requires
dbus-daemon, and all it needs is a random hex string, so
dbus-uuidgen is fine.
Comment 2 Colin Walters 2016-06-07 17:31:41 UTC
Review of attachment 329317 [details] [review]:

LGTM, thanks.
Comment 3 Simon McVittie 2016-06-07 18:08:47 UTC
Comment on attachment 329317 [details] [review]
tests: use dbus-uuidgen instead of uuidgen

01f7ed7
Comment 4 Simon McVittie 2016-06-07 18:09:13 UTC
Fixed in git for 1.45.4