Skip to content
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

runstatedir is not set by autoconf #2267

Closed
metalefty opened this issue May 17, 2022 · 2 comments · Fixed by #2268
Closed

runstatedir is not set by autoconf #2267

metalefty opened this issue May 17, 2022 · 2 comments · Fixed by #2268

Comments

@metalefty
Copy link
Member

After #2207, configure script relies on runstatedir but it is not set in autoconf 2.69.

socketdir and sesmanruntimedir are placed /xrdp and /xrdp-sesman.

I think a workaround like this is needed:
troglobit/sysklogd#6

$ ./configure

xrdp will be compiled with:

  mp3lame                 no
  opus                    no
  fdkaac                  no
  jpeg                    no
  turbo jpeg              no
  rfxcodec                yes
  painter                 yes
  pixman                  no
  fuse                    no
  ipv6                    no
  ipv6only                no
  vsock                   no
  auth mechanism          PAM
  rdpsndaudin             no

  with imlib2             no

  development logging     no
  development streamcheck no

  strict_locations        no
  prefix                  /usr/local
  exec_prefix             ${prefix}
  libdir                  ${exec_prefix}/lib
  bindir                  ${exec_prefix}/bin
  sysconfdir              /etc
  localstatedir           /var
  runstatedir
  socketdir               /xrdp
  sesmanruntimedir        /xrdp-sesman

  unit tests performable  yes

  CFLAGS = -g -O2 -Wall -Wwrite-strings
  LDFLAGS =
@matt335672
Copy link
Member

Good catch. I'll install autoconf 2.69 and have a look.

@metalefty
Copy link
Member Author

The latest autoconf version available on FreeBSD is still 2.69.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants