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

Can't build from 0.42 tarball #4470

Open
spth opened this issue Jun 29, 2024 · 1 comment
Open

Can't build from 0.42 tarball #4470

spth opened this issue Jun 29, 2024 · 1 comment
Labels
pending-verification This issue is pending verification and/or reproduction

Comments

@spth
Copy link

spth commented Jun 29, 2024

Version

The 0.42 tarball (can't build, so can't give yosys --version otuput)

On which OS did this happen?

Linux

Reproduction Steps

tar -xzf yosys-yosys-0.42.tar.gz
cd yosys-yosys-0.42/
LANG=C make

Expected Behavior

yosys builds (like it does for me when using the 0.40 or 0.42 tarballs).

Actual Behavior

yosys fails to build, giving this erroer message:

Initialize the submodule: Run 'git submodule update --init' to set up 'abc' as a submodule.
make: *** [Makefile:748: check-git-abc] Error 1
@spth spth added the pending-verification This issue is pending verification and/or reproduction label Jun 29, 2024
@KrystalDelusion
Copy link
Member

Related to #4403 and the same as #4403 (comment) wherein downloading via the release tar the abc directory is empty and the yosys directory is not a git repository, which also means the git submodule update --init command does not work.

KrystalDelusion added a commit that referenced this issue Jul 1, 2024
This should fail with the current makefile.
Essentially a test case for #4470.
KrystalDelusion added a commit that referenced this issue Jul 19, 2024
This should fail with the current makefile.
Essentially a test case for #4470.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending-verification This issue is pending verification and/or reproduction
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants