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

buildrpm.sh build failed with internel libevent: Invalid version (double separator '-'): 2.1.12-stable-omp #12469

Closed
xin3liang opened this issue Apr 16, 2024 · 8 comments
Assignees

Comments

@xin3liang
Copy link

xin3liang commented Apr 16, 2024

Thank you for taking the time to submit an issue!

Background information

What version of Open MPI are you using? (e.g., v4.1.6, v5.0.1, git branch name and hash, etc.)

branch: v5.0.x commit 1f87690

Describe how Open MPI was installed (e.g., from a source/distribution tarball, from a git clone, from an operating system distribution package, etc.)

If you are building/installing from a git clone, please copy-n-paste the output from git submodule status.

(.venv) [root@client1 ompi]# git submodule status
 33e93469e1e1f69904ff3e3827394719aa6b3671 3rd-party/openpmix (v5.0.2)
 3a70fac9a21700b31c4a9f9958afa207a627f0fa 3rd-party/prrte (v3.0.5)
 dfff67569fb72dbf8d73a1dcf74d091dad93f71b config/oac (heads/main)

Please describe the system on which you are running

  • Operating system/version:
(.venv) [root@client1 ompi]# lsb_release -a
LSB Version:    n/a
Distributor ID: openEuler
Description:    openEuler release 22.03 (LTS-SP3)
Release:        22.03
Codename:       LTS-SP3

  • Computer hardware: x86_64
  • Network type: IB

Details of the problem

when building rpm with buildrpm.sh it fails

git submodule update --init --recursive
diff --git a/VERSION b/VERSION
index 4388a23ce5..1361d960da 100644
--- a/VERSION
+++ b/VERSION
@@ -58,7 +58,7 @@ repo_rev=
 # tarball_version is not empty, the tarball filename will be of the
 # form openmpi-<tarball_version>.tar.*.

-tarball_version=gitclone
+#tarball_version=gitclone
./autogen.pl && ./configure && make dist
./contrib/dist/linux/buildrpm.sh -r "--define 'all_external_3rd_party 0'" -c "--with-lustre" -R ~/ompi/build/  -sbmo ./openmpi-5.0.4a1.tar.gz
...
RPM build errors:
    File listed twice: /opt/openmpi/5.0.4a1/share/doc/openmpi
    File listed twice: /opt/openmpi/5.0.4a1/share/openmpi/modulefiles
    File listed twice: /opt/openmpi/5.0.4a1/share/openmpi/modulefiles/openmpi
    File listed twice: /opt/openmpi/5.0.4a1/share/openmpi/modulefiles/openmpi/5.0.4a1
    File listed twice: /opt/openmpi/5.0.4a1/share/doc/openmpi
    File listed twice: /opt/openmpi/5.0.4a1/share/openmpi/modulefiles
    File listed twice: /opt/openmpi/5.0.4a1/share/openmpi/modulefiles/openmpi
    File listed twice: /opt/openmpi/5.0.4a1/share/openmpi/modulefiles/openmpi/5.0.4a1
    Invalid version (double separator '-'): 2.1.12-stable-ompi: pkgconfig(libevent) = 2.1.12-stable-ompi
    Invalid version (double separator '-'): 2.1.12-stable-ompi: pkgconfig(libevent_core) = 2.1.12-stable-ompi
    Invalid version (double separator '-'): 2.1.12-stable-ompi: pkgconfig(libevent_extra) = 2.1.12-stable-ompi
    Invalid version (double separator '-'): 2.1.12-stable-ompi: pkgconfig(libevent_pthreads) = 2.1.12-stable-ompi
*** FAILURE BUILDING SINGLE RPM!
Aborting

...
@wenduwan
Copy link
Contributor

Likely due to this commit 61e72f9

@wenduwan
Copy link
Contributor

We will look into this. Shouldn't be a hard fix.

@wenduwan wenduwan self-assigned this Apr 24, 2024
@wenduwan
Copy link
Contributor

@xin3liang I tried to reproduce this issue on Amazon Linux 2023 following the provided steps but couldn't get the error. I wonder in particular how did you generate openmpi-5.0.4a1.tar.gz?

@xin3liang
Copy link
Author

xin3liang commented Apr 26, 2024

@xin3liang I tried to reproduce this issue on Amazon Linux 2023 following the provided steps but couldn't get the error. I wonder in particular how did you generate openmpi-5.0.4a1.tar.gz?

Hi @wenduwan thanks for spending time on this issue, please remember to download the git submodules git submodule update --init --recursive
I generate the tarball via commenting out the tarball_version, otherwise, you can replace it with openmpi-gitclone.tar.gz.

diff --git a/VERSION b/VERSION
index 4388a23ce5..1361d960da 100644
--- a/VERSION
+++ b/VERSION
@@ -58,7 +58,7 @@ repo_rev=
 # tarball_version is not empty, the tarball filename will be of the
 # form openmpi-<tarball_version>.tar.*.

-tarball_version=gitclone
+#tarball_version=gitclone

@wenduwan
Copy link
Contributor

@xin3liang I can finally reproduce the problem. Here is a fix for this particular issue #12501

However, I would recommend installing the external libevent for your distro, instead of using the internal libevent.

@wenduwan
Copy link
Contributor

@xin3liang What is your system libevent version? Is it possible to pre-install libevent on your hosts and avoid ompi's internal libevent?

@xin3liang
Copy link
Author

@xin3liang What is your system libevent version? Is it possible to pre-install libevent on your hosts and avoid ompi's internal libevent?

It is libevent-2.1.12-7. I can compile openMPI with the OS provided libevent.

@wenduwan
Copy link
Contributor

wenduwan commented May 6, 2024

@xin3liang Thanks for confirming. In this case we strongly recommend using the system libevent instead.

For the record, we have stated this in our documentation https://docs.open-mpi.org/en/v5.0.x/installing-open-mpi/packagers.html#do-not-use-open-mpi-s-internal-dependent-libraries

@wenduwan wenduwan closed this as completed May 6, 2024
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

3 participants