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

check-git-abc in Makefile for the first compile time #4449

Open
nlwmode opened this issue Jun 13, 2024 · 5 comments
Open

check-git-abc in Makefile for the first compile time #4449

nlwmode opened this issue Jun 13, 2024 · 5 comments
Labels

Comments

@nlwmode
Copy link

nlwmode commented Jun 13, 2024

Version

0.41

On which OS did this happen?

Linux

Reproduction Steps

  1. git clone --recursive https://github.com/YosysHQ/yosys.git
  2. cd yosys
  3. mkdir build && cd build
  4. make -j 128

Expected Behavior

Build successful.

Actual Behavior

Error: 'abc' is not configured as a git submodule.";
To resolve this:";

  1. Back up your changes: Save any modifications from the 'abc' directory to another location.";
  2. Remove the existing 'abc' directory: Delete the 'abc' directory and all its contents.";
  3. Initialize the submodule: Run 'git submodule update --init' to set up 'abc' as a submodule.";
  4. Reapply your changes: Move your saved changes back to the 'abc' directory, if necessary.";
@nlwmode nlwmode added the pending-verification This issue is pending verification and/or reproduction label Jun 13, 2024
@mmicko
Copy link
Member

mmicko commented Jun 13, 2024

@nlwmode

Last step in this case should be make -C .. -j 128 since makefile is in directory above, but that still does not explain issue you get.
Could you please give us info about output of git --version on your machine ?

@nlwmode
Copy link
Author

nlwmode commented Jun 13, 2024

@nlwmode

Last step in this case should be make -C .. -j 128 since makefile is in directory above, but that still does not explain issue you get. Could you please give us info about output of git --version on your machine ?

"git version 2.42.0".

The PR (#4450) carefully explains and solves the problem. Please checking that.

@RCoeurjoly
Copy link
Contributor

I cannot reproduce

This is what I have done:

git clone --recursive https://github.com/YosysHQ/yosys.git yosys_4449
cd yosys_4449/
mkdir build && cd build
make -C ..

I have git version 2.43.1
Can you execute these commands to see if you still have the issue?

@kittennbfive
Copy link
Contributor

Hi,

i can confirm that something is wrong and the output of make install (do this, do this) does not work.

I tried this:

$ cd /tmp
$ git clone --recursive https://github.com/YosysHQ/yosys.git yosys_4449
$ cd yosys_4449/
$ mkdir build && cd build
$ make -C .. -j12
$ su -
# cd /tmp/yosys_4449
# make install

and got:

Error: 'abc' is not configured as a git submodule.
To resolve this:
1. Back up your changes: Save any modifications from the 'abc' directory to another location.
2. Remove the existing 'abc' directory: Delete the 'abc' directory and all its contents.
3. Initialize the submodule: Run 'git submodule update --init' to set up 'abc' as a submodule.
4. Reapply your changes: Move your saved changes back to the 'abc' directory, if necessary.

(But as i said doing so does not solve the issue.)

I am using git version 2.39.2 on Debian stable (12.5 currently).

Workaround
At Makefile:770 remove the entire "if" and replace by exit 0; . Installation works and Yosys works fine too.

Thank you!

@janrinze
Copy link

#4450 appears to work for me on Debian 12.
Could this be a Debian specific issue?

@widlarizer widlarizer added bug and removed pending-verification This issue is pending verification and/or reproduction labels Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants