Skip to content

Comments

Qt6 qpdf#58

Open
alecn2002 wants to merge 5 commits intofhackenberger:masterfrom
alecn2002:qt6-qpdf
Open

Qt6 qpdf#58
alecn2002 wants to merge 5 commits intofhackenberger:masterfrom
alecn2002:qt6-qpdf

Conversation

@alecn2002
Copy link

Hi, I've ported QTikz to under QT 6.3 with QtPdf, removing dependency on Poppler.
It still have some issues, but it works, compiles and shows resulting puicture.
Unfortunately I was able to compile it under Linux only, AFAIK Windows version of QtPdf is not ready yet.

Alexander

@jfmcarreira
Copy link
Collaborator

Does it maintains compatibility with Qt5 in this branch? Otherwise I would suggest us to create a Qt6 branch

@alecn2002
Copy link
Author

alecn2002 commented May 7, 2022

I've tried to do "#if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)" everywhere where I've done changes and keep QT5 code under "#else" branch, but I didn't try to compile under QT5.
And taking into account that it is compilable under Linux only, yes, I think it's a good idea to have separate branch for it.

@jfmcarreira
Copy link
Collaborator

I pushed your branch into this repository with the name qt6.

Is that ok with you? I still need time to check it at least in Linux. On mac i was not able to get the dependencies

@alecn2002
Copy link
Author

Thank you Joao. Yes it's OK for me.
I'm going to continue my development with QPDF usage: in this change I've used conversion to graphics, while QPDF has it's own QPdfViewer widged what eliminates need to generate PNG or other graphic representation - its needed only for exporting to graphic format.
Re: Mac: when you've installed QT 6.3 on Mac, are you sure that you've installed QPdf? It's treated as "experimental" and should be installed separately.
Also I'm not sure if it's available on Mac at all. On Windows it's not available: while there is such an option in QT Installer, only documentation is installed but no libraries, so building under Windows is now impossible. I can't check how things are going on Mac, I don't have access to one.

@alecn2002
Copy link
Author

alecn2002 commented Oct 11, 2022 via email

@jfmcarreira jfmcarreira force-pushed the master branch 9 times, most recently from 9e13f1c to ad8b505 Compare April 27, 2024 22:59
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.

2 participants