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

Provide non-Debian setup instructions and tooling #524

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

vidbina
Copy link

@vidbina vidbina commented Mar 12, 2024

Started this because I was trying to look into the issue I reported (see arXiv/html_feedback#644) and then realised that I couldn't use the Makefile because it was written to be used on Debian (and Debian-derivative) systems (with its use of apt and its fetching of "linux" variants of packages). These changes parameterise the configuration a bit and also provide a more hands-off experience with the use of devenv.sh.

Hopefully this makes it easier for other folks (non-Debian Linux users and even macOS users) to quickly check out the codebase for issues, debug things and contribute patches. It's been parameterised in a way that we should still have the same UX for Debian users as they had prior to the changes. If I failed at this attempt, yell at me for a fix and I'll get on it.

  • chore: Parameterise platform to get system-compat cloud-sql-proxy
  • chore: Parameterise bin path
  • chore: Track devenv gitignore and lockfile
  • chore: Track devenv.nix and yaml
  • chore: Set BIN_PATH and install cloud-sql-proxy in devenv
  • chore: Run poetry install in devenv
  • chore: Gitignore bin
  • doc: Outline non-Debian setup and devenv setup

@vidbina vidbina marked this pull request as ready for review March 12, 2024 15:17
@vidbina
Copy link
Author

vidbina commented Mar 12, 2024

I can understand if this is a bit too instrusive. I stumbled into an issue on arXiv last week and then figured I'd try post a fix but then ran into issues just getting the env to work since I'm on macOS. Hope that the changes are still considered halfway useful to help other macOS or non-Debian users to get up to speed quickly and start contributing to this project.

Why devenv.sh: I really tried to keep devenv out but when I stumbled into the clang and grpcio issues (mentioned in the README), I realised that it would be a considerable timesync and poor UX to write-up a series of imperative steps that one would have to execute in order to get things to work. Declarative looked much clearer to me at that point. Even at the cost of introducing a new tool.

Love arXiv BTW! 🏆 Definitely in my top 5 sites on the interwebs, so I was happy to spend a few minutes tinkering here. 😄

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.

1 participant