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

build: Remove an explicit swig option -ruby #1795

Merged

Conversation

ppisar
Copy link
Contributor

@ppisar ppisar commented Oct 22, 2024

Building Ruby binding with Swig 4.3.0 failed:

cd /home/test/dnf5/redhat-linux-build/bindings/ruby/libdnf5 && /usr/bin/cmake -E env SWIG_LIB=/usr/share/swig/4.3.0 /usr/bin/swig -ruby -DSWIGWORDSIZE64 -module advisory -ruby -outdir /home/test/dnf5/redhat-linux-build/bindings/ruby/libdnf5 -c++ -module advisory -I/home/test/dnf5/include -I/home/test/dnf5/common -I/usr/include -o /home/test/dnf5/redhat-linux-build/bindings/ruby/libdnf5/CMakeFiles/ruby_advisory.dir/advisoryRUBY_wrap.cxx /home/test/dnf5/bindings/libdnf5/advisory.i
Only one target language can be supported at a time (both -ruby and -ruby were specified).

The cause was an explcitit and unnecessary "-ruby" option in CMAKE_SWIG_FLAGS.

This patch removes it. The targetted langauge is already set with swig_add_library().

Resolves: #1794

Building Ruby binding with Swig 4.3.0 failed:

    cd /home/test/dnf5/redhat-linux-build/bindings/ruby/libdnf5 && /usr/bin/cmake -E env SWIG_LIB=/usr/share/swig/4.3.0 /usr/bin/swig -ruby -DSWIGWORDSIZE64 -module advisory -ruby -outdir /home/test/dnf5/redhat-linux-build/bindings/ruby/libdnf5 -c++ -module advisory -I/home/test/dnf5/include -I/home/test/dnf5/common -I/usr/include -o /home/test/dnf5/redhat-linux-build/bindings/ruby/libdnf5/CMakeFiles/ruby_advisory.dir/advisoryRUBY_wrap.cxx /home/test/dnf5/bindings/libdnf5/advisory.i
    Only one target language can be supported at a time (both -ruby and -ruby were specified).

The cause was an explicit and unnecessary "-ruby" option in
CMAKE_SWIG_FLAGS.

This patch removes it. The targetted langauge is already set with
swig_add_library().

Resolves: rpm-software-management#1794
@kontura kontura added this pull request to the merge queue Oct 23, 2024
Merged via the queue into rpm-software-management:main with commit 0dc0063 Oct 23, 2024
20 checks passed
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 this pull request may close these issues.

Ruby binding fails to build with swig-4.3.0-1.fc42
2 participants