Bug 139330 - [PATCH] Handle all possible GValue types
[PATCH] Handle all possible GValue types
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: usermode (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jindrich Novy
David Lawrence
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2004-11-15 04:45 EST by Miloslav Trmač
Modified: 2013-07-02 19:03 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2004-12-17 03:58:14 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Use lu_value_strdup () (1.59 KB, patch)
2004-11-15 04:46 EST, Miloslav Trmač
no flags Details | Diff

  None (edit)
Description Miloslav Trmač 2004-11-15 04:45:56 EST
The libuser API uses GValues without explicitly describing
what GValue types are applicable, and the old G_TYPE_STRING,
G_TYPE_LONG pair is not enough to represent data (UID/GID
in particular) anymore.

This patch uses a new libuser utility function (available since
0.53-1) instead of explicitly handling select GValue types
(and aborting on others).

It also avoids an "unused variable" warning when compiling
without startup-notification.  BTW, startup-notification-devel
should probably be in BuildRequires:
Comment 1 Miloslav Trmač 2004-11-15 04:46:41 EST
Created attachment 106701 [details]
Use lu_value_strdup ()
Comment 2 Jindrich Novy 2004-11-16 04:04:50 EST
The same problem as in bug 139331:

userhelper.o(.text+0x246d): In function `main':
/home/jnovy/CVS_elvis/usermode.build/userhelper.c:1471: undefined
reference to `lu_value_strdup'
userhelper.o(.text+0x2bbc):/home/jnovy/CVS_elvis/usermode.build/userhelper.c:1571:
undefined reference to `lu_value_strdup'
collect2: ld returned 1 exit status
Comment 3 Jindrich Novy 2004-11-16 05:09:12 EST
Fixed, commited, thanks.

Note You need to log in before you can comment on or make changes to this bug.