[okular] [Bug 398338] Compilation failure on Musl due to use of vasprintf

2024-03-30 Thread Jonáš Vidra
https://bugs.kde.org/show_bug.cgi?id=398338

--- Comment #3 from Jonáš Vidra  ---
> The error was fixed upstream in https://github.com/jlaurens/synctex/pull/39
> but reappeared, because the fix was only applied to one branch.

It was actually applied to all branches except for one, which was the one I
checked. An import of the changes from either the `2024` stable branch or the
`main` development one should fix the issue.

-- 
You are receiving this mail because:
You are watching all bug changes.

[okular] [Bug 398338] Compilation failure on Musl due to use of vasprintf

2024-03-30 Thread Jonáš Vidra
https://bugs.kde.org/show_bug.cgi?id=398338

Jonáš Vidra  changed:

   What|Removed |Added

 CC||vidra.jo...@seznam.cz
 Ever confirmed|0   |1
 Status|RESOLVED|REOPENED
 Resolution|UPSTREAM|---

--- Comment #2 from Jonáš Vidra  ---
This bug is still present in Okular 23.08.5.

The error was fixed upstream in https://github.com/jlaurens/synctex/pull/39 but
reappeared, because the fix was only applied to one branch. I filed a new
upstream bug here: https://github.com/jlaurens/synctex/issues/80

There are two related pull requests currently open:
https://invent.kde.org/graphics/okular/-/merge_requests/722
https://invent.kde.org/graphics/okular/-/merge_requests/723

-- 
You are receiving this mail because:
You are watching all bug changes.

[okular] [Bug 398338] Compilation failure on Musl due to use of vasprintf

2018-09-09 Thread Albert Astals Cid
https://bugs.kde.org/show_bug.cgi?id=398338

Albert Astals Cid  changed:

   What|Removed |Added

 Resolution|--- |UPSTREAM
 CC||aa...@kde.org
 Status|UNCONFIRMED |RESOLVED

--- Comment #1 from Albert Astals Cid  ---
Those files are "copied" from syntex, you should open an issue at
https://github.com/jlaurens/synctex/issues since that way we can import fixes
from them.

Also given that musl is kind of not easy to come by i guess it'd be great if
you could provide a patch. It should not be very hard since there seems to be
already an own implementation of vasprintf 

https://github.com/jlaurens/synctex/blob/2017/synctex_parser.c#L8409

Please if you open a bug on their side please link it here so we can also keep
track of it.

-- 
You are receiving this mail because:
You are watching all bug changes.