Bug 222191 - Merge Review: eclipse - An open, extensible IDE
Summary: Merge Review: eclipse - An open, extensible IDE
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: eclipse
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mat Booth
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-01-10 20:14 UTC by Ben Konrath
Modified: 2018-04-11 10:03 UTC (History)
18 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-02-12 09:18:05 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
rplmint output (18.78 KB, text/plain)
2013-01-12 22:27 UTC, Andrey
no flags Details

Description Ben Konrath 2007-01-10 20:14:18 UTC
I'd like to get the eclipse spec file reviewed in preparation for the core and extras merge. I won't be able to commit this to the extras repository because java-gcj-compat uses the eclipse-ecj sub-package but I would still like to go through the process.

Spec URL:

http://people.redhat.com/bkonrath/eclipse/eclipse.spec

SRPM URL:

http://people.redhat.com/bkonrath/eclipse/eclipse-3.2.1-28.fc7.src.rpm

Note that this spec file is already in Fedora cvs and I will be committing changes as I go through this review process.

Description:

The Eclipse Platform is designed for building integrated development
environments (IDEs) that can be used to create applications as diverse
as web sites, embedded Java(tm) programs, C++ programs, and Enterprise
JavaBeans(tm).

Comment 1 Andrew Overholt 2007-01-10 20:17:26 UTC
I wonder if it would be a conflict of interest if I took this one?  Probably not.

Comment 2 Dennis Gilmore 2007-01-10 20:19:10 UTC
only if you dont give it a fair review 

Comment 3 Andrew Overholt 2007-01-10 20:20:28 UTC
I'll give it a fair review.  So I guess I can take this one.  I'm intimately
familiar with the spec so it'll probably be easier for me.

Comment 4 Ben Konrath 2007-01-12 22:19:08 UTC
Updated files are located here:

http://www.bagu.org/eclipse/eclipse.spec
http://www.bagu.org/eclipse/eclipse-3.2.1-30.fc7.src.rpm

Comment 5 Kevin Fenzi 2007-01-24 03:58:39 UTC
Hey Andrew... if you are reviewing this I assume it should be blocking FE-
REVIEW instead of FE-NEW. I am going to change it to do so. If I am in error, 
feel free to change it back. 

Comment 6 Andrew Overholt 2007-01-24 04:57:16 UTC
You are correct about the blocker but Tom's actually going to review it.  I've
re-assigned to him.  Thanks.

Comment 7 Ben Konrath 2007-06-07 04:05:07 UTC
The files referenced in comment #0 are out of date. Please let me know when
somebody is ready to do the review so that I can post updated files. Thanks, Ben

Comment 8 Jason Tibbitts 2008-01-20 01:22:04 UTC
According to the assignment, Thomas is up for this review.  But he hasn't made
any comments in this ticket, so I'm not really sure what's up.

Comment 9 Andrew Overholt 2008-02-07 22:21:42 UTC
I can't speak for Tom, but I'd bet he's completely forgotten about this.  If
someone else can review, I'll work with them on changes as necessary.

Comment 10 Andrey 2009-08-23 10:48:18 UTC
Is this bug still actual ?

Comment 11 Peter Lemenkov 2009-08-23 10:52:08 UTC
(In reply to comment #10)
> Is this bug still actual ?  

Definitely, it is. Nobody made full review so far.

Comment 12 Andrey 2009-08-23 11:49:08 UTC
(In reply to comment #11)

> Definitely, it is. Nobody made full review so far.  

Right now we have F11/Eclipse 3.4.2, initial request was for F7/Eclipse 3.2.1. And I can't see spec file actual for today given somewhere in comments. Link to spec file on comment #4 is dead.

So I still wonder...

Comment 13 Michel Lind 2009-08-23 18:23:40 UTC
It should perhaps be renamed Merge Review, not Review Request anymore.

Comment 15 Alexander Kurtakov 2010-08-24 12:20:23 UTC
what is going on here?
But is assigned to rruss but fedora-review ? flad is set by tibbs.
If there is no response I'll remove both the assigned and fedora-review flag so someone can take the bug and do the review.

Comment 16 Andrew Overholt 2010-08-24 22:26:25 UTC
This is assigned to Rodney because it was assigned to Lillian and he was her manager when she left Red Hat.  I think it can safely be re-assigned and I'll do so now.

Comment 17 Alexander Kurtakov 2010-11-19 14:36:38 UTC
Reset to new.

Comment 18 Andrey 2013-01-03 09:53:49 UTC
This bug is definitely not actual: Core and Extras have been merged thousands of years ago, Eclipse is a part of the default Fedora repo. Why should this bug still be open ?

Comment 19 Matěj Cepl 2013-01-03 19:35:57 UTC
(In reply to comment #18)
> This bug is definitely not actual: Core and Extras have been merged
> thousands of years ago, Eclipse is a part of the default Fedora repo. Why
> should this bug still be open ?

Because nobody ever made that review. This is merge review from times of merging Fedora Core and Fedora Extras (that’s Fedora 7) which nobody ever finished. Do you volunteer to do it?

Comment 20 Andrey 2013-01-03 23:13:36 UTC
I spoke with Mathieu Bridon (bochecha) on QA IRC channel today. It seems I am not a proper person to perform the review as I have no rights to approve the spec neither am I skilled enough for such a complicated package.

We agreed, however, that I will do the 'technical' review - check if all formal 'must-have's are in place and will put it here. Afterwards it will be easier for a real reviewer to finalize the review.

Comment 21 Matěj Cepl 2013-01-04 00:44:58 UTC
(In reply to comment #20)
> I spoke with Mathieu Bridon (bochecha) on QA IRC channel today. It seems I
> am not a proper person to perform the review as I have no rights to approve
> the spec neither am I skilled enough for such a complicated package.
> 
> We agreed, however, that I will do the 'technical' review - check if all
> formal 'must-have's are in place and will put it here. Afterwards it will be
> easier for a real reviewer to finalize the review.

Right, anybody who can write a comment to bugzilla, can create a review. Even if it is not binding, it will greatly help somebody who has the rights (me, for example) to just run through the review, check it, and make the review swiftly and with less effort spent. Also, it greatly counts towards you abilities (and qualification to be packager with proper rights).

Comment 22 Alexander Kurtakov 2013-01-04 09:00:57 UTC
As the maintainer I would be really thankfull if someone does the review. We always wanted to finalize it but it was just not right someone of the current maintainers to do it.

Comment 23 Andrey 2013-01-04 09:13:33 UTC
@Alexander Kurtakov: will you please attach actual spec and SPRM to this bug ? Just to follow the formalities of the Review process: https://fedoraproject.org/wiki/Package_Review_Process#Contributor

Comment 24 Alexander Kurtakov 2013-01-04 09:17:58 UTC
Andrey, this is not the procedure for merge reviews. See https://fedoraproject.org/wiki/Merge_Reviews (Citation: You should be able to review the package against the current HEAD revision in rawhide git. You can use the current rawhide package to check against. )
The justification behind that is that packages already in fedora are updated while the review is going so no static spec/srpm can be attached.

Comment 25 Alexander Kurtakov 2013-01-04 09:19:28 UTC
To make it easier:
* Link to git repo http://pkgs.fedoraproject.org/cgit/eclipse.git/ - contains all patches and spec
* Link to koji page http://koji.fedoraproject.org/koji/packageinfo?packageID=183 - contains all builds of eclipse

Comment 26 Andrey 2013-01-12 22:27:22 UTC
Created attachment 677579 [details]
rplmint output

rpmlint output attached. Small summary of findings is below. I suggest biggest portion of those should be ignored due to specifics of java packages, but I am at the moment just making a summary for further, more experienced reviewers (some warnings can be really important).

0. eclipse-src
0.1. invalid-url (2 times)

1. eclipse-swt.i686
1.1. unstripped-binary-or-object (6 times)
1.2. no-documentation
1.3. hidden-file-or-dir (3 times)

2. eclipse-equinox-osgi.i686
2.1. only-non-binary-in-usr-lib
2.2. no-documentation

3. eclipse-platform.i686
3.1. unstripped-binary-or-object (2 times)
3.2. non-etc-or-var-file-marked-as-conffile
3.3. conffile-without-noreplace-flag (2 times)
3.4. no-documentation
3.5. dangling-symlink (65 times)
3.6. hidden-file-or-dir (13 times)
3.7. incorrect-fsf-address (6 times)
3.8. non-conffile-in-etc
3.9. zero-length
3.10 no-manual-page-for-binary

4. eclipse-jdt
4.1. only-non-binary-in-usr-lib
4.2. no-documentation
4.3. hidden-file-or-dir
4.4. dangling-symlink (3 times)
4.5. dangling-relative-symlink
4.6. no-manual-page-for-binary
4.7. dangerous-command-in-%pre

5. eclipse-pde
5.1. only-non-binary-in-usr-lib
5.2. no-documentation
5.3. dangling-symlink (2 times)
5.4. hidden-file-or-dir
5.5. no-manual-page-for-binary
5.6. invalid-url

6. eclipse-tests
6.1. no-documentation
6.2. script-without-shebang (2 times)
6.3. dangling-symlink
6.4. no-manual-page-for-binary

Comment 27 Cole Robinson 2015-02-11 20:35:26 UTC
Mass reassigning all merge reviews to their component. For more details, see this FESCO ticket:

  https://fedorahosted.org/fesco/ticket/1269

If you don't know what merge reviews are about, please see:

  https://fedoraproject.org/wiki/Merge_Reviews

How to handle this bug is left to the discretion of the package maintainer.

Comment 28 Alexander Kurtakov 2015-02-12 09:18:05 UTC
Eclipse package is going through heavy inspection by current maintainers for every major release (once per year) and there was no interest from anybody stepping up to do official review - thus closing.


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