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

Fix generate_console_pap.py script #51429

Closed
wants to merge 1 commit into from
Closed

Conversation

nyalldawson
Copy link
Collaborator

This script was still using Qt4 QApplication constructor.

I'm not sure how/whether this script is still in use. @jef-n do you have any insights here? How are the api files generated for the release packages now?

@github-actions github-actions bot added this to the 3.30.0 milestone Jan 10, 2023
@m-kuhn
Copy link
Member

m-kuhn commented Jan 21, 2023

Are all these hardcoded paths in this script reasonable?

@jef-n
Copy link
Member

jef-n commented Jan 21, 2023 via email

@jef-n
Copy link
Member

jef-n commented Jan 21, 2023 via email

@nyalldawson
Copy link
Collaborator Author

Thanks @jef-n !

Looks like that's not needed at all anymore. The console builds the pap
from the apis at runtime.

Does this work for you in a custom build? I think there's something broken with the non-release builds there. I'll look into it, but I'm curious if it's working for you...

@nyalldawson
Copy link
Collaborator Author

@jef-n

Looks like that's not needed at all anymore. The console builds the pap from the apis at runtime.

I don't think this is actually the case anymore -- there's logic in the "console_compile_apis.py" to prepare a pap file, but that's all manual user operations and not done by default.

By default we are only hitting https://github.com/qgis/QGIS/blob/master/src/gui/codeeditors/qgscodeeditorpython.cpp#L122 , which loads the file from https://github.com/qgis/QGIS/blob/master/python/qsci_apis/pyqgis.pap, which hasn't been updated in 4 years...

jef-n added a commit that referenced this pull request Jan 25, 2023
@nyalldawson nyalldawson deleted the pap branch January 25, 2023 05:47
jef-n added a commit to jef-n/QGIS that referenced this pull request Jan 25, 2023
jef-n added a commit that referenced this pull request Jan 25, 2023
@YoannQDQ
Copy link
Contributor

Out of curiosity, how are the input .api files created? They seem obsolete.

@nyalldawson
Copy link
Collaborator Author

@YoannQDQ
see #51576 (comment)

domi4484 pushed a commit to domi4484/QGIS that referenced this pull request Jan 27, 2023
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 this pull request may close these issues.

None yet

4 participants