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

make env weirdness #111

Open
rachelfenn opened this issue Feb 3, 2025 · 3 comments
Open

make env weirdness #111

rachelfenn opened this issue Feb 3, 2025 · 3 comments

Comments

@rachelfenn
Copy link
Collaborator

I've had a problem with make env saying there is nothing to be done when in fact there is something that needs to be done.

I wonder if the problem is that nothing has changed in the repo, so the make command believes that there are no changes to be built, but what I need to use make env for is installing. I often see this on the very first install - I run make env, it says nothing to be done, and then the dependencies don't get installed unless I run make -B env.

Perhaps we should split up the install commands and the build command?

@sritchie
Copy link
Contributor

sritchie commented Feb 3, 2025

Hey @rachelfenn , @benlebrun is currently on a big push for GenLM so I'm fielding PRs for now. Do you want to give this a go? LLM-assisted would be my go-to as I'm not Make-literate these days.

@rachelfenn
Copy link
Collaborator Author

Yes, I can implement this, I was just looking for approval on the direction from Ben. This isn't urgent and I can chat with him when he's got time.

@benlebrun
Copy link
Collaborator

@rachelfenn see #94 ; you can use the make refresh_env target. I think the problem is that the genparse.egg-info directory already exists.

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

3 participants