gpe-base/gpe-login has hard coded min UID for users to he considered as valid/real users to 100, which works for many systems but in Gentoo we start valid/real users with UID of 1000 and GID of 1000. The proposed patch adds ./configure switches --with-min-valid-uid=N and --with-min-valid-gid=N that allows tunning the UID and GID values during compilation (defaulting to 100/100). This fixes the display of only valid/real users during in the combobox during login, as well as, makes gpe-login add new users with the right UID and GID. Assigning to myself and waiting gpe-login maintainer's comments for commit. Reproducible: Always
Created attachment 226217 [details, diff] gpe-login-min-uid.patch
(In reply to comment #0) > gpe-base/gpe-login has hard coded min UID for users to he considered as > valid/real users to 100, which works for many systems but in Gentoo we start > valid/real users with UID of 1000 and GID of 1000. > > The proposed patch adds ./configure switches --with-min-valid-uid=N and > --with-min-valid-gid=N that allows tunning the UID and GID values during > compilation (defaulting to 100/100). Looks good. I'm going to point out this bug to upstream. > > This fixes the display of only valid/real users during in the combobox during > login, as well as, makes gpe-login add new users with the right UID and GID. > > Assigning to myself and waiting gpe-login maintainer's comments for commit. Those hardcoded IDs were definitely not the way to go in Gentoo. Thanks for reporting and fixing. > > Reproducible: Always >
> Looks good. I'm going to point out this bug to upstream. Already pinged upstream (Philip Blundell) on this bug, lets hope he responds quickly.
Patch committed to CVS. Upstream says they are going to include this in next version bump. So, we will use this patch until new version is out.
