Project

General

Profile

Actions

Feature #1056

closed

libuuid - non-portabile configure.in

Added by Anonymous about 17 years ago. Updated 2 months ago.

Status:
Fixed
Priority:
Low
Category:
mod_webdav
Target version:
ASK QUESTIONS IN Forums:
No

Description

The current configure.in depends on the presence of a libuuid that is compatible with the e2fsprogs implementation. It assumes 1) the header is /uuid/uuid.h; and 2) uuid_unparse is the function needed to get the string rep of a uuid.

This doesn't work for the OSSP implementation, which puts the header by default in /usr/local/include and uses uuid_export to get at the string representation of a uuid. See http://www.ossp.org/pkg/lib/uuid/.

The OpenBSD ports collection includes the OSSP implementation.

ToDo: 1) fix configure.in to either check for either lib, or to support a --with-uuid=implementation or the like.

Bug: currently if uuid/uuid.h is found, HAVE_UUID_H is #defined. It should be HAVE_UUID_UUID_H.

2) fix mod_webdav.c #includes:

l 23: #if defined(HAVE_LIBXML_H) && defined(HAVE_SQLITE3_H) && defined(HAVE_UUID_H)
will have to branch on HAVE_UUID_H (OSSP implementation) and HAVE_UUID_UUID_H (e2fsprgs implementation). Or maybe define HAVE_OSSP_UUID, HAVE_E2FSPRGS_UUID instead.

2) fix mod_webdav.c usage

line 2301: uuid_unparse(id, uuid);
OSSP does it this way: uuid_export(id, UUID_FMT_STR, (void **)&uuid, NULL);

Proposal: to avoid another #ifdef in the code, have configure.in create a parameterized UUID2STR macro that will substitute the approprite function call.

Actions #1

Updated by darix about 17 years ago

did you test that the OSSP library is working?
can you provide a patch?

Actions #2

Updated by gstrauss about 8 years ago

  • Description updated (diff)
  • Status changed from New to Need Feedback
  • Assignee deleted (jan)

http://www.ossp.org/pkg/lib/uuid/ loads, but any attempt to download the source or view the online doc currently results in an error.

Is there still interest in getting this working? It looks fairly straightforward to put together a patch, but I'll need help from someone who can verify that it works on their platform of choice.

Actions #3

Updated by gstrauss almost 8 years ago

  • Status changed from Need Feedback to Missing Feedback
  • Priority changed from Normal to Low

Is there still interest in getting this working? It looks fairly straightforward to put together a patch, but I'll need help from someone who can verify that it works on their platform of choice.

Actions #4

Updated by gstrauss 2 months ago

  • Status changed from Missing Feedback to Patch Pending
  • Target version set to 1.4.74
  • ASK QUESTIONS IN Forums set to No

mod_webdav now implements a simple function to create a UUID v4, and removes the dependency on libuuid.

Actions #5

Updated by gstrauss 2 months ago

  • Status changed from Patch Pending to Fixed
Actions

Also available in: Atom