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

llvm12.0.1 #6616

Closed
wants to merge 15 commits into from
Closed

llvm12.0.1 #6616

wants to merge 15 commits into from

Conversation

tcanabrava
Copy link
Contributor

@tcanabrava tcanabrava commented Aug 1, 2021

This is an updated version of the stalled MR #1450 that fixes all the current comments and updates the llvm for 12.0.1

I do not want to steal @Manu343726 work, but the last commit on the branch is for more than a year ago. Just trying to update the tools and see if I can help this pass.

Manu343726 is welcome to rebase his work on top of mine and continue.

I haven't tested the build:

➜  llvm git:(llvm) ✗ conan create . test/llvm
ERROR: Conanfile not found at /Users/bl029-user/Projects/conan-center-index/recipes/llvm/conanfile.py

but that folder contains a config.yml and an all folder, that contains the conanfile.
(perhaps the docs should be updated to this new packaging style?)


  • 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.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@tcanabrava
Copy link
Contributor Author

How do I block conan to run the CI on system that have an older gcc than the minimum for this package?

@SSE4
Copy link
Contributor

SSE4 commented Aug 2, 2021

How do I block conan to run the CI on system that have an older gcc than the minimum for this package?

def validate(self):
    if tools.Version(self.settings.compiler) < "5.1":
        raise ConanInvalidConfiguration("give up")

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Failure in build 7 (997bae28f3f699b9cfe8577d3c300afd13efcd99):

  • llvm/12.0.1@:
    CI failed to create some packages (All logs)

    Logs for packageID 2918a38a6bba3f2acf1c7781aff604f376d61b4e:
    [settings]
    arch=x86_64
    arch_build=x86_64
    build_type=Debug
    compiler=apple-clang
    compiler.libcxx=libc++
    compiler.version=12.0
    os=Macos
    os_build=Macos
    
    ********************************************************************************
    conan test conan-center-index/recipes/llvm/all/test_package/conanfile.py llvm/12.0.1@ --profile=/Users/jenkins/w/BuildSingleReference@4/27290/e21616ab-643b-4101-bc51-ef7a618d20af/profile.txt
    ********************************************************************************
    Auto detecting your dev setup to initialize the default profile (/Users/jenkins/w/BuildSingleReference@4/.conan/profiles/default)
    Found apple-clang 12.0
    Default settings
    	os=Macos
    	os_build=Macos
    	arch=x86_64
    	arch_build=x86_64
    	compiler=apple-clang
    	compiler.version=12.0
    	compiler.libcxx=libc++
    	build_type=Release
    *** You can change them in /Users/jenkins/w/BuildSingleReference@4/.conan/profiles/default ***
    *** Or override with -s compiler='other' -s ...s***
    
    
    Configuration:
    [settings]
    arch=x86_64
    arch_build=x86_64
    build_type=Debug
    compiler=apple-clang
    compiler.libcxx=libc++
    compiler.version=12.0
    os=Macos
    os_build=Macos
    [options]
    [build_requires]
    [env]
    
    llvm/12.0.1 (test package): Installing package
    Requirements
        llvm/12.0.1 from local cache - Cache
    Packages
        llvm/12.0.1:2918a38a6bba3f2acf1c7781aff604f376d61b4e - Download
    
    Installing (downloading, building) binaries...
    llvm/12.0.1: Retrieving package 2918a38a6bba3f2acf1c7781aff604f376d61b4e from remote 'c3i_PR-6616' 
    ERROR: Binary package not found: 'llvm/12.0.1:2918a38a6bba3f2acf1c7781aff604f376d61b4e'
    

Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

@tcanabrava
Copy link
Contributor Author

@Manu343726 @SSE4 some help here, from what I can understand from the error the build succeeded, but there was a problem fetching the binaries. I have no clue on what to do now.

@tcanabrava
Copy link
Contributor Author

ping?

@tcanabrava
Copy link
Contributor Author

ping 2?

Comment on lines +59 to +61
tools.get(**self.conan_data["sources"][self.version])
extracted_dir = 'llvm-project-llvmorg-' + self.version
os.rename(extracted_dir, self._source_subfolder)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
tools.get(**self.conan_data["sources"][self.version])
extracted_dir = 'llvm-project-llvmorg-' + self.version
os.rename(extracted_dir, self._source_subfolder)
tools.get(**self.conan_data["sources"][self.version],
destination=self._source_subfolder, strip_root=True)

This will restart the build, which should be enough to make the error go away.

@stale
Copy link

stale bot commented Oct 1, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Oct 1, 2021
@ghost ghost mentioned this pull request Oct 8, 2021
@tcanabrava
Copy link
Contributor Author

a new pull request with llvm 13 was started, closing this.

@tcanabrava tcanabrava closed this Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants