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

bin/configure assumes Debian based system #1435

Closed
coder2000 opened this issue Apr 27, 2024 · 4 comments · Fixed by #1478
Closed

bin/configure assumes Debian based system #1435

coder2000 opened this issue Apr 27, 2024 · 4 comments · Fixed by #1478

Comments

@coder2000
Copy link

I use Arch Linux so my package manager is pacman or yay if I need to use something from AUR. The configure script as provided by the repo assumes that dpkg is available wants to install postgresql and redis system wide. While that isn't an issue in and of itself I prefer to use docker to run them on a per project basis if needed.

@coder2000
Copy link
Author

Splitting it up didn't resolve the issue. The issue is that the scripts rely primarily on dpkg and apt-get which are only available on Debian and derivatives.

@jagthedrummer
Copy link
Contributor

@coder2000 Can you please post a stack trace or some details about whatever problem you ran into? The new scripts no longer rely on dpkg or apt-get for detecting postgres or redis. It should attempt to use dpkg for detecting icu4c but if your system doesn't have dpkg it should just tell you that it knows you don't have dpkg and then it should suggest that you ensure that icu4c is installed.

else
puts "You don't have dpkg, so we can't tell if `icu4c` is installed.".red
puts "Please make sure that `icu4c` is installed.".yellow
end

What exactly are you seeing?

@coder2000
Copy link
Author

Ok. Sorry, I missed that part of the script.

@jagthedrummer
Copy link
Contributor

@coder2000 if it gives you any problems please report back with some details. We definitely want to make this as smooth as possible.

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 a pull request may close this issue.

2 participants