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

libsass: fix build requirements versions conflicts #5597

Conversation

SpaceIm
Copy link
Contributor

@SpaceIm SpaceIm commented May 21, 2021

Specify library name and version: lib/1.0

automake now requires autoconf/2.71, but this recipe "build requires" libtool/2.4.6 (which drags automake) and autoconf/2.69.
Problem: build requirements can't override transitive dependencies, so build fails.

Therefore, I recommend to only require the "top level recipe" in build requirement, here libtool, to minimize maintenance burden.


  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

SpaceIm added 2 commits May 21, 2021 19:29
versions of transitive build requirements can't be overriden, so just rely on libtool to drag autoconf through automake and avoid versions conflicts
@conan-center-bot
Copy link
Collaborator

All green in build 1 (1f2c0356028510c577f4917e700d00485f5b28a3):

  • libsass/3.6.4@:
    All packages built successfully! (All logs)

@SSE4 SSE4 requested a review from uilianries May 22, 2021 07:11
@SSE4
Copy link
Contributor

SSE4 commented May 22, 2021

@madebr can you take a look? I guess you know the relationship of libtool/autoconf/automake better

@conan-center-bot conan-center-bot requested a review from czoido May 27, 2021 11:59
@conan-center-bot conan-center-bot merged commit 7a18764 into conan-io:master Jun 3, 2021
@SpaceIm SpaceIm deleted the fix/libsass-build-requirements-conflicts branch June 3, 2021 10:11
madebr pushed a commit to madebr/conan-center-index that referenced this pull request Jun 21, 2021
* no os.rename

* fix conflicts in build requirements

versions of transitive build requirements can't be overriden, so just rely on libtool to drag autoconf through automake and avoid versions conflicts
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.

5 participants