Bug 783611 - copy-platform should not use $where when symlinking in for loops
Summary: copy-platform should not use $where when symlinking in for loops
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: eclipse
Version: rawhide
Hardware: x86_64
OS: Linux
unspecified
urgent
Target Milestone: ---
Assignee: Andrew Robinson
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-21 00:57 UTC by Severin Gehwolf
Modified: 2012-02-23 13:54 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-23 13:54:58 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 781209 0 unspecified CLOSED FTBFS 2021-02-22 00:41:40 UTC

Internal Links: 781209

Description Severin Gehwolf 2012-01-21 00:57:09 UTC 
Description of problem:
bug 781209 shows the symptom.

I'll attach a patch which fixes bug 781209. Is this $where still needed? Note that at the top of the script there is a cd $where.

I assume one needs either the $where prefix in the for loops or the cd $where, but not both. Thoughts?
Comment 1 Andrew Overholt 2012-01-23 15:26:18 UTC 
Andrew, ISTR you cleaning copy-platform up a bit recently.  Please take a look at this.
Comment 2 Andrew Robinson 2012-01-23 19:56:21 UTC 
Makes sense to me. This was just a straight port of the existing spec generated script which didn't use relative paths. Removing the instances of $where should not be an issue.
Comment 3 Andrew Overholt 2012-01-23 21:01:30 UTC 
Sounds good, please integrate and commit Severin's change, Andrew.
Comment 4 Alexander Kurtakov 2012-02-11 06:01:39 UTC 
Andrew, Severin,
Is this fixed?
If yes please close the bug otherwise please apply the patch submit new build and verify.
Comment 5 Alexander Kurtakov 2012-02-23 07:50:48 UTC 
Is this still an issue ?
Comment 6 Andrew Robinson 2012-02-23 13:54:58 UTC 
The upstream fix is in rawhide.
Note You need to log in before you can comment on or make changes to this bug.