Skip to content

feat(Makefile): Add fusesoc-core-file target; Fix release-artifacts.#324

Merged
jjts merged 3 commits intoIObundle:mainfrom
arturum1:main
Nov 18, 2025
Merged

feat(Makefile): Add fusesoc-core-file target; Fix release-artifacts.#324
jjts merged 3 commits intoIObundle:mainfrom
arturum1:main

Conversation

@arturum1
Copy link
Copy Markdown
Contributor

Add 'fusesoc-core-file' target that allows exporting a single <corename>.core file with a provider URL pointing to the releases page. When FuseSoC is run for this core, it will automatically download the sources from the provider URL.
This file can be submitted to https://github.com/fusesoc/fusesoc-cores repository.

Fix release-artifacts target to avoid including the 'fusesoc_exports' as a sub-directory inside the tar.gz file.

Add 'fusesoc-core-file' target that allows exporting a single
<corename>.core file with a provider url pointing to the releases page.
When FuseSoC is run for this core, it will automatically download the
sources from the provider url.

Fix release-artifacts target to avoid including the 'fusesoc_exports' as
a subdirectory inside the tar.gz file.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new fusesoc-core-file target to generate a standalone .core file with a provider URL pointing to GitHub releases, allowing FuseSoC to automatically download sources. It also fixes the release-artifacts target to avoid including the fusesoc_exports directory as a subdirectory in the generated tar.gz files.

  • Adds fusesoc-core-file target to generate a .core file with remote provider URL
  • Fixes tar command in release-artifacts to properly archive contents without nesting directories
  • Updates clean target to remove generated *.core files

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Makefile Outdated
Comment thread Makefile
define MULTILINE_TEXT
provider:
name: url
url: https://github.com/IObundle/iob-cache/releases/latest/download/$(CORE_NAME)_V$(VERSION).tar.gz
Copy link

Copilot AI Nov 17, 2025

Choose a reason for hiding this comment

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

The GitHub repository URL is hardcoded as 'IObundle/iob-cache'. Consider extracting this as a variable (e.g., REPO_URL or GITHUB_REPO) to make the Makefile more reusable for other projects or repositories.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor Author

@arturum1 arturum1 Nov 17, 2025

Choose a reason for hiding this comment

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

The provider URL can remain hard-coded since all releases of the iob_cache core are in the iob-cache repository.

Comment thread Makefile Outdated
arturum1 and others added 2 commits November 17, 2025 23:38
Remove comment about multiline string usage in Makefile.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jjts jjts merged commit 3103546 into IObundle:main Nov 18, 2025
2 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.

4 participants