Bug 204707 - libuser (and system-config-users) do not support useradd or libuser.conf home directory definitions
Summary: libuser (and system-config-users) do not support useradd or libuser.conf home...
Alias: None
Product: Fedora
Classification: Fedora
Component: system-config-users
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nils Philippsen
QA Contact:
Keywords: FutureFeature
Depends On:
TreeView+ depends on / blocked
Reported: 2006-08-30 22:40 UTC by John E. Koontz
Modified: 2008-08-14 15:36 UTC (History)
2 users (show)

Clone Of:
Last Closed: 2008-08-14 15:36:03 UTC

Attachments (Terms of Use)
Use libuser's default for home directory (970 bytes, patch)
2006-11-02 00:24 UTC, Miloslav Trmač
no flags Details | Diff

Description John E. Koontz 2006-08-30 22:40:42 UTC
FutureFeature applies?

This really applies to both libuser and the overlying system-config-users.  

Description of problem:

As implemented libuser code (and system-config-users code using it) do not
forward information on the default location of user home directories from
useradd's /etc/default/useradd (HOME=) or libuser's /etc/libuser.conf
(LU_HOMEDIRECTORY=).  Useradd does take note of /etc/default/useradd.
The code in system-config-users module userWindow.py uses a constant string:

    def on_newUserNameEntry_focus_out_event(self, *args):
        name = self.userWinUserName.get_text()
        self.userWinHomeDir.set_text("/home/%s" % name)

The code in libuser's config.c, etc., simply indicates that HOME-related
features are unimplented.  

Version-Release number of selected component (if applicable):

All from Red Hat Linux 7.3 to Fedora Core 5, to the extent I have checked. 
Probably all since code was written?  

How reproducible:

Set HOME in /etc/defaults/useradd or LU_HOMEDIRECTORY in /etc/libuser.conf and
open system-user-config.  Assume the setting is appropriate for /local/home.
Actual results:

The new home directory is always in /home, which is perfectly consistent with
the code snippet above, and not surprising given that snippet, etc.  

Expected results:

The new home directory will be in /local/home.  

Additional info:

Because of the explicitness of the coding, and the references to
"unimplemented," there may be some unstated external interaction issue or
planning consideration.  The actual implementation does not look like it would
be particularly involved.  It could be cloned from the code present for other
parameters, with some modifications to remove "%n" etc. from the end of the

Comment 1 Miloslav Trmač 2006-09-08 22:11:01 UTC
As far as I remember it is unimplemented simply because the variables have sligtly
different semantics (the necessary /%n at the end).

s-c-users should get the default value of userWinHomeDir by creating a temporary
user entity with the desired login name and reading its HOMEDIRECTORY attribute.

Comment 2 John E. Koontz 2006-09-11 15:34:46 UTC
I'm getting way out of my depth here, and so my apologies to the knowledgeable 
and experienced.  Still, wouldn't it be easier (and less prone to awkward 
states in the event of a crash) to read adduser's HOME string and trim off the 
%n?  It's true that this assumes knowledge of the internal semantics of 
adduser, but I'm not clear that the process sketched above relies on adduser 
to explain them.  Perhaps the adduser HOME-access could be encapsulated 
somehow to document the fact that it is reaching into adduser?  Most 
convenient would be to modify adduser to provide the root of HOME in some 
fashion.  I imagine it's impractical to change HOME itself.  

Comment 3 Miloslav Trmač 2006-09-11 17:23:51 UTC
Creating a temporary user entity does not really create an user in the system
databases; it only creates an in-memory object.

I think this is the most natural way to get the default value, because it ensures
the default value will be exactly the same as when really creating the user.
Adding a separate interface increases the risk that the two implementations
will eventually diverge due to some oversight.

Comment 4 Miloslav Trmač 2006-11-02 00:24:41 UTC
Created attachment 140059 [details]
Use libuser's default for home directory

default/useradd:HOME importing added in libuser-0.54.8.

The attached patch adds the s-c-u part.

Comment 5 Nils Philippsen 2007-11-07 14:12:53 UTC
Mirek, I've applied your patch in the upstream repo. As FC5 is dead, I'll change
the product version to devel.

Comment 6 Nils Philippsen 2008-08-14 15:36:03 UTC
This should be long fixed.

Note You need to log in before you can comment on or make changes to this bug.