Skip to content

Commit 41a958f

Browse files
elderingvmcj
authored andcommitted
Set the default domjudge_user to the current user at configure time.
Before, the use of `m4_esyscmd` would set the default to the current user at the time configure is generated, which typically happens when we create a release (or snapshot). The `id` command used should be POSIX portable. Closes #1980 (cherry picked from commit 6450b81)
1 parent 55865df commit 41a958f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

configure.ac

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ AC_ARG_WITH([domjudge-user], [AS_HELP_STRING([--with-domjudge-user=USER],
6565
[User that owns password files (default: current user).])], [], [])
6666

6767
if test "x$with_domjudge_user" = x; then
68-
user=m4_esyscmd([id -un])
68+
user="$(id -un)"
6969
# Check and warn if running as root (without explicitly setting it):
7070
if test "x$user" = xroot; then
7171
AC_MSG_ERROR([installing/running as root is STRONGLY DISCOURAGED, use --with-domjudge-user=root to override.])

0 commit comments

Comments
 (0)