Not a problem.

Thanks for the feedback.

:-)

Nito
On 09/15/14 12:02, Mike Gabriel wrote:
Control: notfound -1 3.5.0-27
Control: found -1 3.5.0.27

Hi Nito,

On  Mo 15 Sep 2014 00:28:51 CEST, Nito Martinez wrote:

Package: nx-libs
Version: 3.5.0-27

It would be desirable to add some versions to be able to get the version of the
NXComp library.
These should be defined in NX.h, to be used from other clients, and

Attached is a patch that defines the following functions:

extern const char* NXVersion();
extern int NXMajorVersion();
extern int NXMinorVersion();
extern int NXPatchVersion();

In the patch assigned the VERSION has been set to 3.5.28, just to highlight that
it would be good to change the VERSION of nxcomp whenever a patch is applied
to the code.

Regards,

Nito

Thanks for sending in that patch.

However, the next upstream release of NX redistributed is planned to be 3.5.0.28.

The idea starting the NX version with 3.5.0 was to reflect what "upstream" version we obtained from NoMachine (they use three digits, NX redistributed uses _four_ digits in the version string). The fourth part of the version quadruple is the NX/X2Go/QVD redistributed patch level, so to say.

If that will not cause any technical issues, can you rewrite that patch so it handles four digit version strings (major, minor, micro,  X2Go/QVD-patchlevel).

Please also note that an upstream version like 3.5.0-28 will cause convention issues with most Linux distributions. In Debian, for example, the part left of the "-" denotes the upstream version, the part right of the "-" denotes the package revision inside Debian. So we should avoid using a dash in the NX-libs upstream version.

Thanks!
Mike