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

UUID support configure is a mess (drop libuuid support?) #2

Open
rla opened this issue Jul 3, 2015 · 3 comments
Open

UUID support configure is a mess (drop libuuid support?) #2

rla opened this issue Jul 3, 2015 · 3 comments

Comments

@rla
Copy link

rla commented Jul 3, 2015

With libuuid installed:

checking for uuid_create in -lossp-uuid... no
checking for uuid_create in -luuid... no
WARNING:  Cannot find libossp-uuid or libuuid -- dropping uuid.pl

OSSP-uuid installed, libuuid not:

checking for uuid_create in -lossp-uuid... no
checking for uuid_create in -luuid... yes

as OSSP-uuid installs uuid.h:

./shtool install -c -m 644 uuid.h /usr/local/include/

There is no uuid_create function in libuuid. On Debian, SWI depends on libossp-uuid. libuuid "support" could be dropped maybe or at least get rid of "checking for uuid_create in -luuid" from configure output as it is confusing as hell. As https://github.com/SWI-Prolog/packages-clib/blob/master/uuid.c uses uuid_create, I have no idea how it has ever worked with libuuid.

For reference:

As OSSP ftp and cvs seem to be badly broken at the moment, the library has to be actually sourced from somewhere else.

@JanWielemaker
Copy link
Member

It only works with OSSP-uuid. All UUID libraries I checked have different APIs :( This works fine AFAIK in Debian based systems, SuSE, Redhat and derivates, MacOS and even MinGW for Windows. So in what setting is this going wrong? Of course, if you want to make the system recognise more UUID libraries, I'm happy to apply the patches.

@rla
Copy link
Author

rla commented Jul 5, 2015

So in what setting is this going wrong?

Users spend lots of time installing libuuid and expecting it to work with SWI build before looking at the actual source and seeing that libuuid cannot be used at all (and when you install OSSP-uuid, then it is recognized as libuuid).

The configure output clearly says:

Cannot find libossp-uuid or libuuid

Which hints that actual libuuid might work here, not only OSSP-uuid-recognized-as-libuuid. I'm looking at the autoconf and config.* files if the configure output message can be improved.

@JanWielemaker
Copy link
Member

On 07/05/15 10:40, Raivo Laanemets wrote:

Which hints that actual libuuid might work here, not only
OSSP-uuid-recognized-as-libuuid. I'm looking at the autoconf and
config.* files if the configure output message can be improved.

I'm always in favour of better errors!

Cheers --- Jan

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