| Summary: | 'equivalent' links don't work for emf transaction and validation | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [WebTools] WTP Releng | Reporter: | David Williams <david_williams> | ||||
| Component: | releng | Assignee: | Tran Le <tranle1> | ||||
| Status: | RESOLVED FIXED | QA Contact: | David Williams <david_williams> | ||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | cbridgha, neil.hauge | ||||
| Version: | 3.10 | Flags: | david_williams:
pmc_approved+
david_williams: pmc_approved? (raghunathan.srinivasan) david_williams: pmc_approved? (naci.dai) david_williams: pmc_approved? (deboer) neil.hauge: pmc_approved+ david_williams: pmc_approved? (kaloyan) cbridgha: pmc_approved+ |
||||
| Target Milestone: | 3.10.0 | ||||||
| Hardware: | PC | ||||||
| OS: | Linux | ||||||
| Whiteboard: | PMC | ||||||
| Attachments: |
|
||||||
|
Description
David Williams
I think these are yours Tran? (I know its confusing, and poorly documented ... for me too :) Thanks David, I can work on this issue shortly. Created attachment 196671 [details]
Proposed patch
Patch ready for review.
The patch looks reasonable to try. I suggest you release (and update builder tag) when you are ready to try a build (e.g. after RC2 is declared). Committing "releng" (by itself) will not trigger a build (unlike the other releng map files) so you may have to 'manually' press the button on CC page to force a rebuild, if no other fixes are being released. For completeness, marking for PMC Review. Without this fix to "releng files", we'll have to "manually" fix the PHP files, which is a pain, though we could for final release, if anyone would prefer not to have this change, and then fix after the actual release. But risk is minimal ... no change to effect code. At worst would break build or PHP page in which case it'd be easy to back out. (And, only reason there is any uncertainty, is this is the first case of a URL where we have ".../?project=xxxx..." so I think there is some minimal chance something could go wrong with the 'ant replace' substitutions ... but, worth just trying, instead of testing, IMHO. Patch applied and released in RC3. Looks great! Thanks Tran. And ... now that the link works :) ... makes it more obvious there are some "1.5 versions" being contributed to Indigo (from what I can tell) ... so you may want to work with Diagram Editor team to see if that pre-req should be updated to 1.5 RC1? (In reply to comment #7) > may want to work with Diagram Editor team to see if that pre-req should be > updated to 1.5 RC1? I inquired about this last week but have not yet heard back from Stefan. I'll ping him again if I don't hear anything today. (In reply to comment #8) > (In reply to comment #7) > > > may want to work with Diagram Editor team to see if that pre-req should be > > updated to 1.5 RC1? > > I inquired about this last week but have not yet heard back from Stefan. I'll > ping him again if I don't hear anything today. Still haven't heard back from Stefan on this issue. I think that as you mentioned before, users will be getting 1.5 when they download from the Indigo repository, so I think we need to upgrade these dependencies to be in line with the Indigo repo. I'll enter a separate bug for this. |