Opened 20 months ago

Last modified 7 days ago

#1229 assigned Bug / Defect

mingw builds: version number missing in LZO DLL

Reported by: Gert Döring Owned by: Samuli Seppänen
Priority: minor Milestone: release 2.5.3
Component: Building / Compiling Version: OpenVPN git master branch (Community Ed)
Severity: Not set (select this one, unless your'e a OpenVPN developer) Keywords:
Cc: Selva Nair, stipa


as discussed on the hackathon: something in our mingw builds of liblzo is missing the insertion of the version number (as a resource binary) into liblzo.dll.

It works for other stuff we build, so "how hard can it be" :-)

Assigning to Samuli so this is not lost.

Attachments (1)

lzo-details.png (23.7 KB) - added by Selva Nair 11 months ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 19 months ago by tincantech

Test for #1238 -- Do you receive notification ?

comment:2 Changed 11 months ago by Gert Döring

Cc: snair added

This is not critical, but it would be nice if it could be fixed...

@selva, do you have an idea how to tackle that?

comment:3 Changed 11 months ago by Gert Döring

Cc: Selva Nair added; snair removed

Changed 11 months ago by Selva Nair

Attachment: lzo-details.png added


comment:4 Changed 11 months ago by Selva Nair

LZO source does not seem to come with a resource file. We can add one (with result as in the image below), but we'll have to maintain the patch. As lzo hardly gets any updates, it may not be a big burden, though.


comment:5 Changed 9 months ago by Gert Döring

Cc: stipa added

As @stipa does much of the heavy lifting of the windows releases right now, cc'ing

comment:6 Changed 3 months ago by Gert Döring

Milestone: release 2.5release 2.5.3

So, how do we want to proceed here? Improve things (for 2.5.3 or 2.6) or "it's the way it is, compression will eventually go away anyway [in 2.8 or so], so close the ticket"?

comment:7 Changed 3 months ago by stipa

I checked vcpkg lzo port and they (vcpkg port maintainers) also don't bother to set version with custom rc file, so let's leave it as it.

comment:8 Changed 7 days ago by Samuli Seppänen

I say we just ignore this. At least not in our mingw buildsystem, as that's going away in favour of MSVC for the only builds we really care about, that is, Windows builds.

comment:9 Changed 7 days ago by Samuli Seppänen

Priority: majorminor
Note: See TracTickets for help on using tickets.