Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 259715 - x11-libs/libcompizconfig bundles an internal copy of iniparser
Summary: x11-libs/libcompizconfig bundles an internal copy of iniparser
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard: Pending removal: 2018-01-27
Keywords: PMASKED
: 267666 (view as bug list)
Depends on: 267753 275429
Blocks: bundled-libs 278132
  Show dependency tree
 
Reported: 2009-02-20 13:31 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2018-02-14 15:29 UTC (History)
10 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Updated patch to allow libcompizconfig to build against the system iniparser (libcompizconfig-0.8.2-allow-system-libiniparser.patch,36.36 KB, patch)
2009-04-28 10:04 UTC, John Keeping
Details | Diff
Updated patch with only the minimal necessary changes (libcompizconfig-0.8.2-allow-system-libiniparser.patch,6.83 KB, patch)
2009-05-02 14:20 UTC, John Keeping
Details | Diff
Error message from fusion-icon (backtrace libcompizconfig,4.02 KB, text/plain)
2009-07-17 07:38 UTC, Chema
Details
Patch for libcompizconfig-0.8.8.ebuild (libcompizconfig.patch,884 bytes, patch)
2012-08-20 15:55 UTC, Sergey Popov (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2009-02-20 13:31:59 UTC
The package is present in Portage as dev-libs/iniparser, would be a nice idea to get upstream to use the system copy if present so we don't have to add yet another copy of it.
Comment 1 Jorge Manuel B. S. Vicetto (RETIRED) gentoo-dev 2009-04-27 02:03:20 UTC
Fixed on libcompizconfig-0.8.2-r1.

ppc/ppc64 I had to drop your keywords until you keyword dev-libs/iniparser. Please keyword at your convenience.
Comment 2 Diego Elio Pettenò (RETIRED) gentoo-dev 2009-04-27 09:12:59 UTC
Not fixed at all; seems like libcompizconfig uses internal code of the iniparser library, you didn't check that compiz started up after your patch, did you?

Comment 3 Alexander 2009-04-27 13:37:52 UTC
ccsm                                                                
Info: No sexy-python package found, don't worry it's optional.                
Traceback (most recent call last):                                            
  File "/usr/bin/ccsm", line 99, in <module>                                  
    import compizconfig                                                       
ImportError: /usr/lib/libcompizconfig.so.0: undefined symbol: iniparser_setstr


Have a similar problem



* dev-libs/iniparser
     Available versions:  (~)3.0b
Comment 4 ak47wj 2009-04-27 14:13:05 UTC
Can't start compiz-manager after apply the patch, /usr/lib/libcompizconfig.so.0: undefined symbol: iniparser_new
Comment 5 Meconio 2009-04-27 18:18:00 UTC
I've same problem
Comment 6 Alex Bennee 2009-04-27 19:19:42 UTC
Yep r1 is broken.

emerge -v =x11-libs/libcompizconfig-0.8.2

Fixes the problem mentioned
Comment 7 Jorge Manuel B. S. Vicetto (RETIRED) gentoo-dev 2009-04-27 19:21:33 UTC
(In reply to comment #2)
> Not fixed at all; seems like libcompizconfig uses internal code of the
> iniparser library, you didn't check that compiz started up after your patch,
> did you?

I see I broke libcompizconfig.
I'll try to see what I can do to it when I get home.
Comment 8 Jorge Manuel B. S. Vicetto (RETIRED) gentoo-dev 2009-04-27 19:25:06 UTC
(In reply to comment #6)
> Yep r1 is broken.

I've asked Tomas Chvatal (scarabeus) to mask libcompizconfig-0.8.2-r1.
Comment 9 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2009-04-27 21:05:36 UTC
*** Bug 267666 has been marked as a duplicate of this bug. ***
Comment 10 John Keeping 2009-04-28 10:04:10 UTC
Created attachment 189689 [details, diff]
Updated patch to allow libcompizconfig to build against the system iniparser

This patch changes the function names to allow libcompizconfig to build against the system iniparser. Currently there will still be one implicitly defined function (iniparser_set) as the iniparser.h installed by dev-libs/iniparser is inconsistent with the functions defined by iniparser.c in the same package. I'll file a bug for that and make it block this one.
Comment 11 Diego Elio Pettenò (RETIRED) gentoo-dev 2009-05-01 14:09:52 UTC
John, could you please submit a cleaned up patch, skipping autotools regeneration and just providing the actual changes you edited yourself? You should be able to either use a git checkout of the libcompizconfig repository or quilt to get the clean patch.
Comment 12 John Keeping 2009-05-02 14:20:18 UTC
Created attachment 190123 [details, diff]
Updated patch with only the minimal necessary changes

I've updated the patch to include only the necessary changes (the autoconf changes from the patch currently in portage and my changes to the source to allow it to build).

I'm still not 100% happy with this since I've had to knock out iniparser.h completely from the source to ensure that it isn't included, which means that the configure changes to use the system iniparser are more or less redundant since applying this patch will force you to use iniparser.h from your system.
Comment 13 Uros 2009-07-02 18:21:38 UTC
.. could someone please mask 0.8.2-r1? 0.8.2 works. 0.8.2-r1 does not.
Comment 14 Marcello Magaldi 2009-07-02 18:25:19 UTC
(In reply to comment #13)
> .. could someone please mask 0.8.2-r1? 0.8.2 works. 0.8.2-r1 does not.
> 

here on amd64 0.8.2-r2 works great
Comment 15 Uros 2009-07-02 21:15:36 UTC
sorry.. had it unmasked locally.
Comment 16 Jorge Manuel B. S. Vicetto (RETIRED) gentoo-dev 2009-07-16 13:54:18 UTC
I bumped libcompizconfig to 0.8.2-r2 in the overlay with the latest patch - thanks to John Keeping.
Will update the ebuild in the tree shortly.
Comment 17 Jorge Manuel B. S. Vicetto (RETIRED) gentoo-dev 2009-07-16 18:14:37 UTC
I just added the updated ebuild to the tree.

@ppc, @ppc64:

I had to drop your keywords for x11-libs/libcompizconfig-0.8.2-r2 as you're still missing keywords for dev-libs/iniparser-3.0b-r1
Comment 18 Chema 2009-07-17 07:37:25 UTC
I just upgraded to 0.8.2-r2 from the overlay and it's the same, compiz won't start.

If i execute compiz --replace i get: compiz (core) - Error: Couldn't load plugin 'yes'

When executing fusion-icon i get no message, it simply doesn't start, but when i change to kwin i get the error and backtrace of the attachment.
Comment 19 Chema 2009-07-17 07:38:15 UTC
Created attachment 198256 [details]
Error message from fusion-icon
Comment 20 Marcello Magaldi 2009-07-17 07:50:28 UTC
(In reply to comment #19)
> Created an attachment (id=198256) [edit]
> Error message from fusion-icon
> 

also here on x86 compiz doesn't start anymore after the upgrade, I get the following error :
compiz --replace
compiz (core) - Fatal: GLX_EXT_texture_from_pixmap is missing
compiz (core) - Error: Failed to manage screen: 0
compiz (core) - Fatal: No manageable screens found on display :0.0
Comment 21 Marcello Magaldi 2009-07-17 08:09:05 UTC
My problem could be related to xorg ati drivers because on my amd64 machine I have nvidia propertary drivers and on that machine compiz still works also after the upgrade and on other two x86 machines I have ati xorg driver and compiz stopped working with the error I reported above. 
Downgrading libcompizconfig on x86 machines to 0.8.2 makes compiz work again.


Should I open another bug for this ?



Comment 22 Jorge Manuel B. S. Vicetto (RETIRED) gentoo-dev 2009-07-17 22:36:03 UTC
(In reply to comment #21)
> Should I open another bug for this ?

Please do.
Comment 23 Marcello Magaldi 2009-07-19 22:45:08 UTC
(In reply to comment #22)
> (In reply to comment #21)
> > Should I open another bug for this ?
> 
> Please do.
> 

opened bug #278132 
Comment 24 Eitan Mosenkis 2009-07-21 14:59:42 UTC
I have the same error as Chema on 0.8.2-r2 on amd64 / nvidia proprietary drivers.  compiz seems to start, but all settings are wiped and any attempt to change them results in them resetting again.  At one point, I believe I looked at the Default.ini and the format was very different from what it is on 0.8.2, but I've downgraded to fix the problem and don't have a copy of the -r2 ini.
Comment 25 Jorge Manuel B. S. Vicetto (RETIRED) gentoo-dev 2009-07-24 14:16:18 UTC
I forgot to add a comment here that libcompizconfig-0.8.2-r2 has been masked.
I'll keep working on this.
Comment 26 Joe Jezak (RETIRED) gentoo-dev 2009-12-27 08:04:18 UTC
ppc/ppc64 have iniparser marked, please add us back when we're needed.
Comment 27 BRULE Herman 2010-03-02 21:08:54 UTC
Bug too under 0.8.4:
compiz (core) - Error: Couldn't load plugin 'yes'
Comment 28 Sergey Popov (RETIRED) gentoo-dev 2012-08-20 15:55:52 UTC
Created attachment 321770 [details, diff]
Patch for libcompizconfig-0.8.8.ebuild

Bug still presents in libcompizconfig-0.8.8.ebuild

Suggesting patch for ebuild to fix in. Please commit it as 0.8.8-r1
Comment 29 Naohiro Aota gentoo-dev 2012-08-27 15:29:02 UTC
The patch mostly seems fine. However I'm not completely sure libcompizconfig-0.8.2/src/ini.c change. Are there any upstream discussion about this issue?
Comment 30 Sergey Popov (RETIRED) gentoo-dev 2012-08-27 17:09:12 UTC
(In reply to comment #29)
> The patch mostly seems fine. However I'm not completely sure
> libcompizconfig-0.8.2/src/ini.c change. Are there any upstream discussion
> about this issue?

No, i did not find any discussion about that. It seems that upstream thinks that this is normal behaviour
Comment 31 Sergey Popov (RETIRED) gentoo-dev 2012-08-30 15:58:45 UTC
Comment on attachment 321770 [details, diff]
Patch for libcompizconfig-0.8.8.ebuild

It seems that this patch is someway broken - with it ini files are broken and unusuable :-(
Comment 32 Pacho Ramos gentoo-dev 2018-02-14 15:29:29 UTC
removed from the tree