Skip to content

Undefined $SED in configure #10392

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

Closed
albandil opened this issue May 16, 2022 · 2 comments
Closed

Undefined $SED in configure #10392

albandil opened this issue May 16, 2022 · 2 comments

Comments

@albandil
Copy link

albandil commented May 16, 2022

When I configure the v5.0.0rc7 tag from the repository on openSUSE Tumbleweed (with up to date submodules and autogen.pl executed), I see this output:

(...)
*** Fortran compiler
checking whether the compiler supports GNU Fortran... yes
checking whether gfortran-12 accepts -g... yes
checking for BSD- or MS-compatible name lister (nm)... ../../configure: line 32694: 1q: command not found
../../configure: line 32700: 1q: command not found
../../configure: line 32694: 1q: command not found
../../configure: line 32700: 1q: command not found
../../configure: line 32694: 1q: command not found
../../configure: line 32700: 1q: command not found
/usr/bin/nm
checking the name lister (/usr/bin/nm) interface... BSD nm
(...)

The lines 32694 and 32700 are similar and both contain $SED '1q'. The content of the variable $SED is apparently empty. Only when I do

SED=sed ./configure

then the configuration finishes without complaints. Open MPI, PMIX, as well as PRTE seem to be affected by the same problem; I see the same set of command-not-found messages three times.

@ggouaillardet
Copy link
Contributor

Thanks for the report!

some AC_PROG_SED calls are missing in various configure.ac

I also noted lex library detection is busted on Tumbleweed:
/usr/lib64/libfl.so expects a yylex symbol but it is not defined in the conftest.c generated by autotools.

I will issue the PRs and check the flex detection issue later on.

ggouaillardet added a commit to ggouaillardet/ompi that referenced this issue May 17, 2022
Refs. open-mpi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
ggouaillardet added a commit to ggouaillardet/prrte that referenced this issue May 17, 2022
Refs. open-mpi/ompi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
ggouaillardet added a commit to ggouaillardet/pmix that referenced this issue May 17, 2022
Refs. open-mpi/ompi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
rhc54 pushed a commit to openpmix/openpmix that referenced this issue May 17, 2022
Refs. open-mpi/ompi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
ggouaillardet added a commit to ggouaillardet/pmix that referenced this issue May 17, 2022
Refs. open-mpi/ompi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
(cherry picked from commit 34a26e8)
ggouaillardet added a commit to ggouaillardet/ompi that referenced this issue May 18, 2022
Refs. open-mpi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
jsquyres pushed a commit to jsquyres/ompi that referenced this issue May 18, 2022
Refs. open-mpi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
(cherry picked from commit 2012196)
ggouaillardet added a commit to ggouaillardet/prrte that referenced this issue May 19, 2022
Refs. open-mpi/ompi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
@awlauria
Copy link
Contributor

Fixed by #10400 (main) and #10406 (v5.0.x)

rhc54 pushed a commit to openpmix/prrte that referenced this issue May 20, 2022
Refs. open-mpi/ompi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
awlauria pushed a commit to awlauria/ompi that referenced this issue Jun 22, 2022
Refs. open-mpi#10392

Signed-off-by: Gilles Gouaillardet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants