New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Patch for linux LDAP groups #9311
Comments
From ddick@aapt.net.auCreated by ddick@aapt.net.aut/op/groups.t fails when the linux is configured with LDAP authentication and the group has a space in it. This patch adds linux to the list of operating systems with spaces in the group name. Perl Info
|
From ddick@aapt.net.augroups.patchdiff -Naur old/t/op/groups.t new/t/op/groups.t
--- old/t/op/groups.t 2005-07-14 03:43:13.000000000 +1000
+++ new/t/op/groups.t 2008-04-29 09:53:49.796263720 +1000
@@ -134,7 +134,7 @@
print "# gr = @gr\n";
my %did;
-if ($^O =~ /^(?:uwin|cygwin|interix|solaris)$/) {
+if ($^O =~ /^(?:uwin|cygwin|interix|solaris|linux)$/) {
# Or anybody else who can have spaces in group names.
$gr1 = join(' ', grep(!$did{$_}++, sort split(' ', join(' ', @gr))));
} else {
|
From @rgsThanks, applied as #33774 to bleadperl. |
The RT System itself - Status changed from 'new' to 'open' |
@rgs - Status changed from 'open' to 'resolved' |
From @TuxOn Wed, 30 Apr 2008 05:17:54 -0700, David Dick (via RT)
Nice find, but if Linux supports spaces in LDAP groups, wouldn't all Isn't it better/more reliable to somehow find out if it is LDAP based -- |
Migrated from rt.perl.org#53560 (status was 'resolved')
Searchable as RT53560$
The text was updated successfully, but these errors were encountered: