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

build fails on opensuse tumbleweed #732

Open
carrotIndustries opened this issue Aug 29, 2023 · 9 comments
Open

build fails on opensuse tumbleweed #732

carrotIndustries opened this issue Aug 29, 2023 · 9 comments

Comments

@carrotIndustries
Copy link
Member

@fk0815 Looks like opensuse tumbleweed updated libpodofo from 0.9.8 to 0.10 and didn't keep the 0.9 version around.
podofo 0.10 is API-incompatible with the 0.9 series, so all other distros kept 0.9 around in one way or another. What's the best way to reach the opensuse maintainers to keep podofo 0.9 packaged?

@fk0815
Copy link

fk0815 commented Aug 30, 2023

the best, I assume, would be to raise that topic on the opensuse factory mailinglist. I raised the topic there already: https://lists.opensuse.org/archives/list/factory@lists.opensuse.org/thread/B7H6ETA2J5VA4Z63ZJ7UWEL6RGY7IY6D/

@carrotIndustries
Copy link
Member Author

Ok, so far still no 0.9.8 podofo in tumbleweed it seems: https://software.opensuse.org/search?baseproject=openSUSE%3AFactory&q=podofo

carrotIndustries added a commit that referenced this issue Sep 11, 2023
@fk0815
Copy link

fk0815 commented Sep 12, 2023

The mailinglist response was that a podofo098 compatibility package has been kept in tumbleweed until all dependent packages have been updated to be compatible with the latest podofo. Horizon is not in the openSUSE:Factory project, which is feeding tumbleweed, it is in electronics project, therefore it did not trigger to keep the podofo098 package. I could try to submit a podofo098 package to the electronics project, users need then to install this one then to use horizon.
Is there a plan to update horizon for the new podofo API?

@carrotIndustries
Copy link
Member Author

Is there a plan to update horizon for the new podofo API?

Eventually yes, but I once tried to do both with a bunch of ifdefs and the code was miserable.

Supporting both would require effectively maintaining two PDF exporters or writing a bunch of useless wrapper code to paper over the API differences.

Since I prefer not doing either of those, my current plan is staying on podofo 0.9.8 until podofo 0.10 has become available in the oldest supported debian and ubuntu releases.

So if we want to keep Horizon EDA working on tumbleweed, I think the only option is adding 0.9.8 to the electronics project.

@fk0815
Copy link

fk0815 commented Sep 15, 2023

Would it be possible to apply a patch with the new API changes for those distros that do not have the old libpodofo still available?

@carrotIndustries
Copy link
Member Author

Would it be possible to apply a patch with the new API changes for those distros that do not have the old libpodofo still available?

That's more or less writing an maintaining two PDF exporters, just less convenient.

Right now, I don't have the motivation to port the PDF exporter to podofo 0.10. If anyone wants to jump in, I'll probably merge it.

If we want to support both podofos, the way to go would be having two sets of source files that get selected depending on the detected version of podofo.

@fk0815
Copy link

fk0815 commented Sep 25, 2023

I started some experimental work trying to adapt to the new libpodofo API here fk0815@1f8f517, but as I'm not a C++ developer, I got lost a bit in the PDFExporter::export_pdf() implementation to get that compiling. So help or hints are welcome.

@fk0815
Copy link

fk0815 commented Oct 5, 2023

This one fk0815@a99bc22 compiles, does the export, but the output looks somehow incorrect (duplicated text) and annotations are missing, etc. I'm struggling with c++ here an there, @carrotIndustries do you want to have a look on that early version?

@fk0815
Copy link

fk0815 commented Oct 29, 2023

This #755 fixes the build on tumbleweed, but there seems to be a new issue with the windows build that is independent from my changes: #756

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

No branches or pull requests

2 participants