Bug 680999 - Eclipse copy-platform script shouldn't use wildcard for dropins
Summary: Eclipse copy-platform script shouldn't use wildcard for dropins
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: eclipse
Version: 15
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Andrew Overholt
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-02-28 18:14 UTC by Jeff Johnston
Modified: 2011-07-05 19:41 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-07-05 19:41:04 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Proposed patch (796 bytes, patch)
2011-02-28 23:32 UTC, Jeff Johnston
no flags Details | Diff

Description Jeff Johnston 2011-02-28 18:14:50 UTC
Description of problem:

The copy-platform script uses wildcards when checking if the desired components exist in either /usr/lib/eclipse/dropins or /usr/share/eclipse/dropins.  This causes problems when a name is a subset of a dropins folder.  For example, the eclispe-cdt package needs to load rse plugins and so specifies rse in the list passed to copy-platform.  There is a sub-package eclipse-cdt-parsers that installs into /usr/lib/eclispe/dropins/cdt-parsers.  The rse is a subset of the name and so the plugins and features get copied over.  This causes the eclipse-cdt build to not build some of those plugins/features.

The script should only be allowing the given names when searching in the dropins folders.

Version-Release number of selected component (if applicable):

This problem exists on f14, f15, and rawhide.

How reproducible:

Attempt to build Fedora eclipse-cdt package on system where eclipse-cdt-parsers is already installed.

Actual results:

Build will fail.

Expected results:

Build succeeds.

Comment 1 Andrew Overholt 2011-02-28 22:45:48 UTC
Do you have a patch, Jeff?

Comment 2 Jeff Johnston 2011-02-28 23:32:25 UTC
Created attachment 481473 [details]
Proposed patch

Adding a proposed patch.

Comment 3 Alexander Kurtakov 2011-03-14 11:13:12 UTC
Andrew, Jeff, 
Do you think I should include it in next eclipse-build?

Comment 4 Andrew Overholt 2011-03-14 13:49:57 UTC
Jeff's patch is reversed but if it fixes the issue for him (and I understand what he's trying to accomplish and it looks okay to me), yes, let's apply it upstream.  Thanks.

Comment 5 Andrew Overholt 2011-06-29 21:05:01 UTC
I've been looking into making this change upstream but I'm first going to ensure that none of the Eclipse packages in Fedora rely on this globbing.

Comment 6 Andrew Overholt 2011-07-05 15:40:28 UTC
I rebuilt all eclipse-* packages with a locally-modified copy-platform and everything builds fine.  I'll see about getting this change into eclipse-build upstream.

Comment 7 Andrew Overholt 2011-07-05 19:41:04 UTC
This was really an upstream eclipse-build bug and I've fixed it there in commit c5838267623ca6d5de4475227f3c70b977c6147e:

  http://git.eclipse.org/c/linuxtools/org.eclipse.linuxtools.eclipse-build.git/commit/?id=c5838267623ca6d5de4475227f3c70b977c6147e

Watch for it in a future Eclipse build in rawhide.


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