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 692733 - move from /var/run to /run, in accordance with systemd design principle
move from /var/run to /run, in accordance with systemd design principle
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
3.7.x
Other Linux
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2013-01-28 20:13 UTC by Dominique Leuenberger
Modified: 2013-02-20 16:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Move from /var/run to /run (1.84 KB, patch)
2013-01-28 20:13 UTC, Dominique Leuenberger
committed Details | Review
Add a --with-run-dir which sets the default for everything in [/var]/run (2.95 KB, patch)
2013-02-07 18:54 UTC, Simon McVittie
committed Details | Review

Description Dominique Leuenberger 2013-01-28 20:13:01 UTC
Using systemd, we're supposed to occupy /run instead of /var/run.

Thus, changing the defaults sounds sane.
Comment 1 Dominique Leuenberger 2013-01-28 20:13:41 UTC
Created attachment 234652 [details] [review]
Move from /var/run to /run
Comment 2 Ray Strode [halfline] 2013-01-28 20:17:16 UTC
sure, thanks.
Comment 3 Ray Strode [halfline] 2013-02-05 05:30:11 UTC
I had to revert this. it breaks distcheck, and it's not immediately apparent to me why.
Comment 4 Simon McVittie 2013-02-07 18:52:51 UTC
May I suggest having a --with-run-dir configure option, or something? For historical reasons[1], Debian uses [/var]/run/gdm3 instead of just gdm.

[1] for a while we supported both gdm (gdm 2) and gdm3, while Gdm 3 caught up with Gdm 2's feature set; these days we only have Gdm 3
Comment 5 Simon McVittie 2013-02-07 18:54:05 UTC
Created attachment 235436 [details] [review]
Add a --with-run-dir which sets the default for everything  in [/var]/run

Distributions that don't like /var/run can override this to
/run/gdm; distributions where the directory differs (like Debian,
which uses /run/gdm3) can override this and it'll affect everything.
Comment 6 Ray Strode [halfline] 2013-02-07 20:16:35 UTC
this suffers from the same issue.  make uninstall ends up deleting outside of DESTDIR.

We probably could just drop the hooks where we create and destroy the directories at build time, since gdm now creates them at run time.
Comment 7 Simon McVittie 2013-02-12 19:15:28 UTC
(In reply to comment #6)
> this suffers from the same issue.  make uninstall ends up deleting outside of
> DESTDIR.

I believe this only applies if you change the default to be something not starting with ${prefix}. make distcheck uses default settings (unless you change them in DISTCHECK_CONFIGURE_FLAGS) so if you leave the default as ${localstatedir}/run/gdm, as my patch does, it should be fine?

(You could even set the default to /run/gdm, and put --with-run-dir='${localstatedir}/run/gdm' in DISTCHECK_CONFIGURE_FLAGS.)

> We probably could just drop the hooks where we create and destroy the
> directories at build time, since gdm now creates them at run time.

Creating: yes, I think that would be reasonable. /run will be empty after each reboot anyway.

For deleting I'm not sure, it does seem a bit wrong if `make uninstall` leaves droppings behind (even in a transient directory)?
Comment 8 Ray Strode [halfline] 2013-02-20 16:22:25 UTC
okay i'll push this.