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.

Sign up for GitHub

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

Jump to bottom

Article : QGIS 4 (QGIS basé sur Qt6) #1211

Open
wants to merge 34 commits into
base: master
Choose a base branch
from
Open

Article : QGIS 4 (QGIS basé sur Qt6) #1211

wants to merge 34 commits into from

Conversation

Guts
Copy link
Member

@Guts Guts commented Oct 8, 2024

No description provided.

@github-actions github-actions bot added the articles Tutoriels, actualité, billet... label Oct 8, 2024
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
2 similar comments
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Guts Guts force-pushed the article/qgis-4-futur branch from 0e5bfb3 to cb04118 Compare October 9, 2024 09:12
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Guts Guts force-pushed the article/qgis-4-futur branch from cb04118 to 0410c6b Compare October 9, 2024 09:14
@Guts Guts self-assigned this Oct 9, 2024
@Guts
Copy link
Member Author

Guts commented Oct 9, 2024

ping @troopa81 pour ajuster la partie linux et build...

@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
troopa81
troopa81 reviewed Oct 9, 2024
View reviewed changes
content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md Outdated Show resolved Hide resolved
content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md Outdated
cd ~/Git
git clone https://github.com/qgis/QGIS.git -b release-3_38 --single-branch --depth 1
cd QGIS
CXX=clang++-14 && CC=clang-14 && cmakeQGIS -DWITH_QTWEBKIT=FALSE -DWITH_SERVER=FALSE -DBUILD_WITH_QT6=ON -DCMAKE_PREFIX_PATH="$DEPENDS_DIR/qwt/install" -DCMAKE_INSTALL_PREFIX=/usr/local/bin/qgis-build/
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si on build avec clang il faut installer le paquet qui va bien. Sinon, pas forcément la peine de setter ces variables et on utilisera gcc par défaut (amené par build-essential il me semble)

-DCMAKE_PREFIX_PATH="$DEPENDS_DIR/qwt/install"

Cette partie nécessite d'être compilé à la main au préalable j'en ai bien peur. A moins qu'un paquet qwt-qt6 existe enfin?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si on build avec clang il faut installer le paquet qui va bien. Sinon, pas forcément la peine de setter ces variables et on utilisera gcc par défaut (amené par build-essential il me semble)

C'est quoi l'avantage de builder avec clang ? Je pense que moins on ajoute de fioriture, mieux c'est.

Cette partie nécessite d'être compilé à la main au préalable j'en ai bien peur. A moins qu'un paquet qwt-qt6 existe enfin?

Idéalement ce serait apt install qwt-qt6 ? Sinon je sais pas comment faire...

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est quoi l'avantage de builder avec clang ? Je pense que moins on ajoute de fioriture, mieux c'est.

peut-être plus rapide, et encore. Pour un simple build mieux vaut faire sans en effet

Idéalement ce serait apt install qwt-qt6 ? Sinon je sais pas comment faire...

Je vois tjs pas de paquet...

tu peux télécharger les sources ici

Sur le site ils disent de build comme ça

cd qwt-6.3.0
qmake6 qwt.pro
make
make install

Ca va installer sur ton système, pas idéal, mais pas grave. Si tu veux pas, tu peux copier le contenu du build (aprés make, pas besoin de make install) comme ça:

 cp -Rf lib/* ./install/lib
 cp -Rf src/* ./install/include/qwt

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai tenu compte de tes remarques et essayé de simplifier au mieux l'installation. On ne peut vraiment pas builder QGIS sans Qwt ?

@troopa81 peux-tu checker que c'est OK et cohérent avec la commande de build de QGIS en suivant stp ?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai tenu compte de tes remarques et essayé de simplifier au mieux l'installation. On ne peut vraiment pas builder QGIS sans Qwt ?

Non. Il existe dans fedora, mais pas encore dans debian/ubuntu...

@troopa81 peux-tu checker que c'est OK et cohérent avec la commande de build de QGIS en suivant stp ?

J'ai pas la bande passante pour faire ça en ce moment

@Guts @troopa81
Update content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-…
517b8f4 
…source.md

Co-authored-by: Julien Cabieces <[email protected]>
Signed-off-by: Julien <[email protected]>
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Guts
Copy link
Member Author

Guts commented Nov 11, 2024

Allez zou, c'est parti pour la @geotribu/relecture !

ℹ️ Aux relecteur/ices

Merci d'utiliser autant que possible le mode Suggestion de la review : https://docs.github.com/fr/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request#starting-a-review

image

C'est vraiment IMPORTANT pour le confort de l'auteur/ice 🙏

@Guts Guts changed the title Article : QGIS 4 (WIP) Article : QGIS 4 Nov 11, 2024
@Guts Guts changed the title Article : QGIS 4 Article : QGIS 4 (QGIS basé sur Qt6) Nov 11, 2024
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
gounux
gounux reviewed Nov 12, 2024
View reviewed changes
Copy link
Contributor

@gounux gounux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pas grand chose à redire, je trouve l'article intéressant et drôle. Merci

content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md Outdated Show resolved Hide resolved
content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md Outdated Show resolved Hide resolved
@Guts @gounux
Update content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-…
41577eb 
…source.md

Co-authored-by: Guilhem Allaman <[email protected]>
Signed-off-by: Julien <[email protected]>
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Guts @nicogodet
Apply suggestions from code review
62cab38 
Co-authored-by: Nicolas Godet <[email protected]>
Signed-off-by: Julien <[email protected]>
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
lbartoletti
lbartoletti reviewed Nov 12, 2024
View reviewed changes
content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md Outdated Show resolved Hide resolved
content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md Outdated Show resolved Hide resolved
content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md Outdated Show resolved Hide resolved
content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md Show resolved Hide resolved
content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-source.md

Derrière le titre _putaclic_, l'article est l'occasion de glisser quelques éléments de compréhension d'un projet logiciel : dépendances, dynamique communautaire, forge logicielle....

On aura aussi vu qu'il y a plusieurs manières d'empaqueter et distribuer le même code source. Ici, OSGeo4W et vcpkg.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oui, il faudrait rappeler que vcpkg est comme un arbre des ports (BSD), mais n'est pas (encore) équivelent à un système de packaging à la pacman d'arch/msys2, pkg des BSD, et autres dnf/yum

@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
1 similar comment
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@Guts @lbartoletti
Update content/articles/2024/2024-10-29_tester-qgis-4-futur-sig-open-…
ddca8d8 
…source.md

Co-authored-by: Loïc Bartoletti <[email protected]>
Signed-off-by: Julien <[email protected]>
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions github-actions bot added the mkdocs Tout ce qui est lié au générateur de site : configuration, hooks, scripts... label Nov 12, 2024
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions
Copy link
Contributor

👏 Le markdown ajouté ou modifié est impeccable ! Merci 🥇

@github-actions GitHub Actions
Copy link
Contributor

🎉 Le site de prévisualisation pour cette PR a été généré et est disponible sous ce lien.
Commit de référence : d1b8f8e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gounux gounux gounux left review comments

@nicogodet nicogodet nicogodet left review comments

@troopa81 troopa81 troopa81 left review comments

@lbartoletti lbartoletti lbartoletti left review comments

@arno974 arno974 arno974 left review comments

@mapperfr mapperfr Awaiting requested review from mapperfr mapperfr was automatically assigned from geotribu/relecture

Assignees

@Guts Guts

Labels
articles Tutoriels, actualité, billet... mkdocs Tout ce qui est lié au générateur de site : configuration, hooks, scripts... outillage Linter, git hooks, CI, CD, build, compilation, devops, intégration et déploiement continus, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Guts @arno974 @lbartoletti @troopa81 @nicogodet @gounux