First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 7531
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Michael Wechner <michael.wechner@wyona.org>
Hardware:
OS:
Version:
Priority:
Severity:
Target Milestone:
Reporter: Michael Wechner <michael.wechner@wyona.org>
Add CC:
CC:
QA Contact:
URL:
Summary:
Status Whiteboard:
Keywords:
Flags:
 

Attachment Type Creator Created Size Flags Actions
fix suggested in comment 2 with some debug logging patch Guillaume Déflache 2010-04-21 13:37 1.87 KB none Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 7531 depends on: Show dependency tree
Show dependency graph
Bug 7531 blocks:

Additional Comments:







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2010-03-18 15:04 
Pleaes see description of the pronblem at
http://lists.wyona.org/pipermail/yanel-development/2010-March/004511.html

------- Comment #1 From Michael Wechner 2010-03-18 15:06:49 ------- 
I think the problem is located within

src/build/java/org/wyona/yanel/ant/MergeResourceTypesConfigsTask.java

------- Comment #2 From Guillaume Déflache 2010-04-21 12:43:12 ------- 
I changed:
if (!resourceTypeExists(srcAttr, rootElement)) {

to:
if (srcAttr == null || srcAttr.equals("") || !resourceTypeExists(srcAttr,
rootElement)) {

and it seems to work as expected with default development build settings for
me.

The problem was that for a "package" resource-type there is no 'src' attribute,
srcAttr gets empty but this also happens for all other "package"
resource-types, so the RT is seen as a duplicate by the 'resourceTypeExists'
method.

AFAICS this is broken since r42497.

------- Comment #3 From Michael Wechner 2010-04-21 13:08:12 ------- 
sounds great :-) Can you attach a patch and I will be happy to test/review it
as well?

Thanks

Michi

------- Comment #4 From Guillaume Déflache 2010-04-21 13:37:16 ------- 
Created an attachment (id=1294) [details]
fix suggested in comment 2 with some debug logging

------- Comment #5 From Guillaume Déflache 2010-04-28 12:33:22 ------- 
I committed the patch in r49096 (debug logging was removed): we needed it badly
for our project! I will monitor Hudson for failures in the next minutes, hope
it's OK that way.
First Last Prev Next    No search results available      Search page      Enter new bug