MerginMaps / mobile

Mobile app for Mergin Maps 📲. Open QGIS projects and manage vector data on your phone, even offline 📶 (Android/iOS/Windows app)
https://merginmaps.com
GNU General Public License v2.0
280 stars 62 forks source link

Fix code layout CI - install previous astyle #3423

Closed tomasMizera closed 1 month ago

tomasMizera commented 1 month ago

New astyle version (3.4.15) did not work well, see https://sourceforge.net/p/astyle/bugs/570/

I reverted it back to 3.4.13, it was not available in homebrew, so I installed it via pip. Not sure if it is a good idea, all of us will need to install it via pip now. Another option is to either build it locally or write custom astyle.rb definition file for homebrew (see https://github.com/Cloudxtreme/ds-cli/blob/0d511740f049834bd5e6872a8878fb237b531a03/platform/mac/homebrew/Library/Formula/astyle.rb#L4)

This way it works at least here! :)

How to install it locally:

python3 -m venv ~/pip3-env # <--- you choose the path
source ~/pip3-env/bin/activate
pip3 install astyle===3.4.13

... use astyle as previously, but you need to have that env activated when using it

3.4.13 is the latest version on pip for now.