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

Running make locally fails - space in dir name #428

Closed
yaronf opened this issue Feb 8, 2024 · 4 comments
Closed

Running make locally fails - space in dir name #428

yaronf opened this issue Feb 8, 2024 · 4 comments

Comments

@yaronf
Copy link

yaronf commented Feb 8, 2024

On a Mac (but with "make" correctly installed)

git clone -q --depth 10  \
    -b main https://github.com/martinthomson/i-d-template lib
lib/venv.mk:233: warning: overriding recipe for target '/Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My'
lib/venv.mk:228: warning: ignoring old recipe for target '/Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My'
lib/venv.mk:233: warning: overriding recipe for target 'Drive/Home'
lib/venv.mk:228: warning: ignoring old recipe for target 'Drive/Home'
lib/venv.mk:233: warning: overriding recipe for target 'Dir'
lib/venv.mk:228: warning: ignoring old recipe for target 'Dir'
lib/venv.mk:273: warning: overriding recipe for target '/Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My'
lib/venv.mk:233: warning: ignoring old recipe for target '/Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My'
lib/venv.mk:273: warning: overriding recipe for target 'Drive/Home'
lib/venv.mk:233: warning: ignoring old recipe for target 'Drive/Home'
lib/venv.mk:273: warning: overriding recipe for target 'Dir'
lib/venv.mk:233: warning: ignoring old recipe for target 'Dir'
lib/venv.mk:272: *** mixed implicit and normal rules: deprecated syntax
make: Circular /Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My <- /Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My dependency dropped.
make: Circular Drive/Home <- /Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My dependency dropped.
make: Circular Drive/Home <- Drive/Home dependency dropped.
make: Circular Dir <- /Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My dependency dropped.
make: Circular Dir <- Drive/Home dependency dropped.
make: Circular Dir <- Dir dependency dropped.
make: Circular 2022-8/misc/JOSE-COSE-Guidance/lib/.venv/bin/.initialized-with-Makefile.venv <- /Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My dependency dropped.
make: Circular 2022-8/misc/JOSE-COSE-Guidance/lib/.venv/bin/.initialized-with-Makefile.venv <- Drive/Home dependency dropped.
make: Circular 2022-8/misc/JOSE-COSE-Guidance/lib/.venv/bin/.initialized-with-Makefile.venv <- Dir dependency dropped.
python3 -m venv /Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My Drive/Home Dir 2022-8/misc/JOSE-COSE-Guidance/lib/.venv
/Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My Drive/Home Dir 2022-8/misc/JOSE-COSE-Guidance/lib/.venv/bin/python -m pip install  --upgrade pip setuptools wheel
bash: /Users/ysheffer/Library/CloudStorage/GoogleDrive-yaron_sheffer@intuit.com/My: No such file or directory
make: *** [lib/venv.mk:229: 2022-8/misc/JOSE-COSE-Guidance/lib/.venv/bin] Error 127

This is all very messy (and leaves several folders behind as it fails), but it looks like the root cause is a shell command that fails to handle a directory called "My Drive" somewhere up the tree.

@yaronf
Copy link
Author

yaronf commented Feb 9, 2024

See sio/Makefile.venv#5

martinthomson added a commit that referenced this issue May 15, 2024
@martinthomson
Copy link
Owner

@yaronf, I took a look at this. I've made some changes that mean that relative paths are used in makefiles. That made the ruby installation a massive pain, but I think that I have successfully sorted out most of the kinks. Let me know if this backfires.

@yaronf
Copy link
Author

yaronf commented May 23, 2024

@martinthomson Works like magic, thanks!

@martinthomson
Copy link
Owner

That's because it is :) I'm happy that it worked out.

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

No branches or pull requests

2 participants