-
Notifications
You must be signed in to change notification settings - Fork 57
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
Update and improve BUILDING.md #239
Comments
Everything in the yml file for ease of understanding. No Docker. For #239
This project uses Docker to build the binaries, and unfortunately this results in the build process not fully self contained/self documenting in https://github.com/AppImageCommunity/AppImageUpdate/blob/main/ci/Dockerfile.x86_64 seems to be the key. (@TheAssassin: For this reason, I always like to have a self-contained .yml file in my projects that build the project, with no other files involved.) |
There is no reason to maintain a building README in my opinion. We don't do that elsewhere either. And I certainly don't want to do it. |
Ohhh thank you, that is good to know. I wouldn't remove these steps from the repository: git clone --recursive https://github.com/AppImage/AppImageUpdate
cd AppImageUpdate/
mkdir build/
cd build/
cmake -DBUILD_QT_UI=OFF -DCMAKE_INSTALL_PREFIX=/usr ..
make -j$(nproc)
sudo make install as they're not self-explanatory and pretty important for people without experience building this project to get that done. |
I agree. I'd rather remove it. In my opinion, the Dockerfile sufficiently documents the necessary requirements for a certain distribution. Some dependencies we build ourselves (e.g., googletest) can be used just fine from the distribution on newer releases.
|
The building markdown file is outdated (CentOS hasn't been maintained in the last three years), incomplete (there is no tutorial for Arch-based distributions) and misses some packages. It seems like the packages someone needed to install on their system were just written down, but if you run the given commands on a fresh Ubuntu instance, many packages miss, e.g. GPGME, nlohmann json, argagg, boost and many more.
It would be good to update it and improve it to include a complete tutorial on how to build it.
The text was updated successfully, but these errors were encountered: