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

Example files are installed by the server build, and additionally they are in a wrong place #54423

Open
1 of 2 tasks
gdt opened this issue Aug 30, 2023 · 0 comments
Open
1 of 2 tasks
Assignees
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Build/Install Related to compiling or installing QGIS Server Related to QGIS server

Comments

@gdt
Copy link
Contributor

gdt commented Aug 30, 2023

What is the bug or the crash?

As a packager I built qgis with qgis server enabled, and the files admin.sld and wms_metadata.xml are installed in ${QGIS_CGIBIN_DIR}. These are not cgi binaries and thus should not be there. As far as I can tell (qgis/QGIS-Documentation#8475 has links) these are examples that someone might want to adapt. Therefore, if they are installed, they should be in $prefix/share/examples/qgis/server, or something like this.

Steps to reproduce the issue

Option 1) Build and install qgis with -DWITH_SERVER:BOOL=TRUE and look in the cgi dir.

Option 2) Read

install(FILES
This can be plainly seen to be inetnional, and the bug is simply that it shouldn't happen.

(I did not try running a new qgis profile, as the bug is clear from make install without ever starting qgis.)

Versions

Observed in 3.28.10, and confirmed in git master with link above.

Supported QGIS version

  • I'm running a supported QGIS version according to the roadmap.

New profile

Additional context

I suggest simply removing these lines from CMakeLists.txt.

@gdt gdt added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label Aug 30, 2023
@nicogodet nicogodet added Build/Install Related to compiling or installing QGIS Server Related to QGIS server labels Aug 31, 2023
@pblottiere pblottiere self-assigned this Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Build/Install Related to compiling or installing QGIS Server Related to QGIS server
Projects
None yet
Development

No branches or pull requests

3 participants