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 Review | Assignee: | Susi Lehtola <susi.lehtola> | ||||
Status: | CLOSED DEFERRED | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | low | ||||||
Version: | rawhide | CC: | 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
Jess Portnoy
2009-03-10 21:24:59 UTC
Hello, Looking for a sponsor for this package. Thanks in advance, 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. Hello Itamar, I've created the account and joined the packager group. My username is kernel01. Thanks, (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} Created attachment 334730 [details]
codelite gcc-4.4 fix
codelite gcc-4.4 fix
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) (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. there are also a folder called sqlite3, you need to get rid of then and include sql-devel in your buildrequires. :-) 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, What is the status of this bug? ping again, someone? Hello, I got a new source release from the maintainer that should be OK. I'll prepare an RPM later this week and update. - 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,-). 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. (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. > 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.]
ping? ping again? Closing due to inactive submitter. |