Bug 1050825

Summary: codeblocks bundles astyle
Product: [Fedora] Fedora Reporter: Rahul Sundaram <metherid>
Component: codeblocksAssignee: Dan Horák <dan>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: dan
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: codeblocks-13.12-3.fc21 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1054422 (view as bug list) Environment:
Last Closed: 2014-04-12 11:37:20 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1054422    
Bug Blocks: 504493    
Attachments:
Description Flags
starting point for building astyle shared library none

Description Rahul Sundaram 2014-01-09 07:16:07 UTC
Description of problem:

astyle is available as a separate package in Fedora but Code::Blocks appears to bundle its own copy.  please unbundle and depend on the astyle package instead.

Comment 1 Dan Horák 2014-01-09 09:15:45 UTC
While the bundled astyle code is likely even not forked, there is no standlone astyle library that could be used for C::B.

Comment 2 Rahul Sundaram 2014-01-09 09:22:58 UTC
I am not sure about the veracity of that assertion given that there is a standalone astyle package in Fedora but in any case, if you need an exception to the fedora packaging guidelines, you need to file a ticket with FPC.  I have done so now

https://fedorahosted.org/fpc/ticket/380

Comment 3 Dominik 'Rathann' Mierzejewski 2014-01-16 18:32:57 UTC
(In reply to Dan Horák from comment #1)
> While the bundled astyle code is likely even not forked, there is no
> standlone astyle library that could be used for C::B.

I'm sorry, but this is simply wrong. astyle upstream explicitly supports building a shared library, so it's a bug in astyle package that it doesn't provide it.

http://astyle.sourceforge.net/install.html#_GCC_Compiler

Please open a bug against astyle and block on it instead of closing.

Comment 4 Dominik 'Rathann' Mierzejewski 2014-01-16 18:35:17 UTC
Created attachment 851219 [details]
starting point for building astyle shared library

Here's a proof-of-concept patch against current rawhide astyle.spec. The shared library needs to have its SONAME set before being consumable properly, but that's an issue for astyle upstream.