Skip to content

add mojo-wayland recipe - #376

Closed
Phelsong wants to merge 2 commits into
modular:mainfrom
Phelsong:main
Closed

Phelsong wants to merge 2 commits into
modular:mainfrom
Phelsong:main

Conversation

@Phelsong

Copy link
Copy Markdown
Contributor

Checklist

  • [ x ] My recipe.yaml file specifies which version(s) of MAX is compatible with my project (see here for an example). If not, my package is compatible with both 24.5 and 24.6.
  • [ x ] License file is packaged (see here for an example).
  • [ x ] Set the build number to 0 (for new packages, or if the version changed).
  • [ x ] Bumped the build number (if the version is unchanged).

new project

@alisterburt

Copy link
Copy Markdown
Collaborator

hey @Phelsong, looks like the build is failing because the license file gets added twice: recipes/mojo-wayland/LICENSE is in the recipe folder and license_file: LICENSE also picks up the one from your repo, so rattler-build errors with File already exists: info/licenses/LICENSE
https://github.com/modular/modular-community/actions/runs/36453375027/job/109033415199?pr=376

should be an easy fix, just dropping the LICENSE from the recipe folder should do it! To make it easier to know what's ready for review I'm closing PRs with failing builds, please reopen when you've had a chance to update this 🙂

@Phelsong

Copy link
Copy Markdown
Contributor Author

I had already removed it, the last build passed.

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.

2 participants