Bug 489598

Summary: Review Request: codelite - a powerful open-source, cross platform code editor for C/C++
Product: [Fedora] Fedora Reporter: Jess Portnoy <kernel01>
Component: Package ReviewAssignee: Susi Lehtola <susi.lehtola>
Status: CLOSED DEFERRED QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: bugs.michael, fedora, fedora-package-review, itamar, lemenkov, notting, susi.lehtola
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-07-22 12:35:16 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
codelite gcc-4.4 fix none

Description Jess Portnoy 2009-03-10 21:24:59 UTC
Spec URL: http://dl.sourceforge.net/sourceforge/codelite/codelite.spec
SRPM URL: http://downloads.sourceforge.net/codelite/codelite-1.0.2785-1.fc10.src.rpm
Description: 
CodeLite uses a sophisticated, yet intuitive interface which allows users to easily create, build and debug complex projects.

Comment 1 Jess Portnoy 2009-03-10 21:26:56 UTC
Hello,

Looking for a sponsor for this package.

Thanks in advance,

Comment 2 Itamar Reis Peixoto 2009-03-10 22:00:45 UTC
do you have a FAS account ?

if not please create one here.

https://fedoraproject.org/wiki/PackageMaintainers/Join#Get_a_Fedora_Account

after account created please join the fedora package group in FAS and post your username here, this will speed up the process.

Comment 3 Jess Portnoy 2009-03-10 22:15:56 UTC
Hello Itamar,

I've created the account and joined the packager group. 
My username is kernel01.

Thanks,

Comment 4 Itamar Reis Peixoto 2009-03-10 23:19:57 UTC
(In reply to comment #3)

ok, now please try to setup koji

https://fedoraproject.org/wiki/PackageMaintainers/Join#Install_the_Client_Tools_.28Koji.29

for some reason your package doesn't build in fedora-rawhide

please take a look in build logs

http://koji.fedoraproject.org/koji/taskinfo?taskID=1235705

after setup koji in your box please try to build with something like this.

koji build dist-f11 codelite-1.0.2785-1.fc10.src.rpm --scratch

also please replace your configure line in your spec file with this

%configure --plugins-dir=%{_libdir}/%{name}

Comment 5 Itamar Reis Peixoto 2009-03-10 23:21:14 UTC
Created attachment 334730 [details]
codelite gcc-4.4 fix 

codelite gcc-4.4 fix

Comment 6 Itamar Reis Peixoto 2009-03-10 23:22:23 UTC
I am still not able to build even with attached patch

here are the build error.


/sdk/wxscintilla/src/scintilla/include -I./sdk/wxpropgrid/include        -I./sdk/wxscintilla/src/scintilla/src -I./Interfaces -I./Debugger -I./Plugin -I./CodeLite  -c LiteEditor/output_pane.cpp -o Release_gcc_unicode/output_pane.o
LiteEditor/output_pane.cpp: In member function 'void OutputPane::CreateGUIControls()':
LiteEditor/output_pane.cpp:77: error: 'wxWindow' is an inaccessible base of 'BuildTab'
LiteEditor/output_pane.cpp:80: error: 'wxWindow' is an inaccessible base of 'ErrorsTab'
make: *** [Release_gcc_unicode/output_pane.o] Error 1
error: Bad exit status from /var/tmp/rpm-tmp.xgd4Hc (%build)

Comment 7 Jess Portnoy 2009-03-11 07:17:10 UTC
(In reply to comment #6)
> I am still not able to build even with attached patch
> 
> here are the build error.
> 
> 
> /sdk/wxscintilla/src/scintilla/include -I./sdk/wxpropgrid/include       
> -I./sdk/wxscintilla/src/scintilla/src -I./Interfaces -I./Debugger -I./Plugin
> -I./CodeLite  -c LiteEditor/output_pane.cpp -o
> Release_gcc_unicode/output_pane.o
> LiteEditor/output_pane.cpp: In member function 'void
> OutputPane::CreateGUIControls()':
> LiteEditor/output_pane.cpp:77: error: 'wxWindow' is an inaccessible base of
> 'BuildTab'
> LiteEditor/output_pane.cpp:80: error: 'wxWindow' is an inaccessible base of
> 'ErrorsTab'
> make: *** [Release_gcc_unicode/output_pane.o] Error 1
> error: Bad exit status from /var/tmp/rpm-tmp.xgd4Hc (%build)  

Hello Itamar,

Many thanks. I applied the patch and tested against GCC 4.4.
I will attach the src.rpm later today.
I've also set up koji late last night so I will use it before posting.

Comment 8 Itamar Reis Peixoto 2009-03-11 12:16:18 UTC
there are also a folder called sqlite3, you need to get rid of then and include sql-devel in your buildrequires.

:-)

Comment 9 Jess Portnoy 2009-03-12 12:16:18 UTC
Status update:

I have a patch that works, also with GCC 4.4 [the error Itamar attached relates to inheretence issues, the reason I didn't catch it is that GCC 4.4 is much stricter, which is a good thing, conclusion: always run with koji], I gave it to the maintainer and I am waiting for him to release a new upstream source tar ball. Also, I asked him to change the name for le_dos2unix.sh, le_exec.sh and le_killproc.sh, these are scripts he supplies and I want them to have a name that indicates that are related to codelite. 
Also, after checking the code, he doesn't really need Curl for anything but Windows so they should not be part of the install target.

I hope to get the upsteam fixes soon, once done, I'll koji it and post.

Thanks,

Comment 10 Mamoru TASAKA 2009-05-14 15:32:21 UTC
What is the status of this bug?

Comment 11 Mamoru TASAKA 2009-05-21 16:39:04 UTC
ping again, someone?

Comment 12 Jess Portnoy 2009-05-25 10:53:43 UTC
Hello,

I got a new source release from the maintainer that should be OK.
I'll prepare an RPM later this week and update.

Comment 13 Susi Lehtola 2009-05-25 12:06:48 UTC
- Use %configure and make %{?_smp_mflags} if you have a very good reason not to (i.e. if they don't work).

- NO STRIPPING!!! This is automatically done by rpm, which produces the -debuginfo package.

- What does --copy-generic-name-to-name do in desktop-file-install? It's not documented.

- Use %defattr(-,root,root,-).

Comment 14 Jess Portnoy 2009-05-25 15:11:57 UTC
I cannot use %configure as CodeLite build is not done by GNU's auto tools.
--copy-generic-name-to-name
  Copy the contents of the GenericName field to the Name field
Its in the man page.
About the strip, these files were not stripped unless I explictly ran this, causing rpmlint to complain.

Comment 15 Susi Lehtola 2009-05-25 16:17:03 UTC
(In reply to comment #14)
> I cannot use %configure as CodeLite build is not done by GNU's auto tools.

OK. Please enable SMP make though, if it works.

> --copy-generic-name-to-name
>   Copy the contents of the GenericName field to the Name field
> Its in the man page.

There is no man page for desktop-file-utils. The only documentation is the --help switch, and it doesn't give that on F10.

> About the strip, these files were not stripped unless I explictly ran this,
> causing rpmlint to complain.  

Stripping yourself causes broken debuginfo packages.

**

- You need at least BuildRequires: desktop-file-utils in order to build in mock (or koji).

- You need to use

make CMP="g++ %{optflags} -fPIC -DON_64_BIT" GCC="gcc %{optflags} -fPIC  -DON_64_BIT"

to make the build process use the optimization flags given by RPM.


- SMP make doesn't seem to work. You should contact upstream about this; not using SMP make is *slow* for a program of this size.

Comment 16 Michael Schwendt 2009-05-25 17:57:41 UTC
> About the strip, these files were not stripped unless I explictly
> ran this, causing rpmlint to complain.  

Install the "redhat-rpm-config" package. [It will hopefully become a dependency of the "fedora-packager" package in the near future.]

Comment 17 Susi Lehtola 2009-06-08 19:03:38 UTC
ping?

Comment 18 Susi Lehtola 2009-07-05 10:35:59 UTC
ping again?

Comment 19 Susi Lehtola 2009-07-22 12:35:16 UTC
Closing due to inactive submitter.