PCTCompileExt "forceCompile" not working

[GoogleCodeExporter]

I noticed that PCTCompileExt wasn't always respecting the last-modified 
timestamps for ".cls" files. It seems to work just fine for PCTCompile.

I created an additional test-case for "PCTCompileExtTest" (see attached).

I'm using OpenEdge 10.2B0406 and PCT v0.17 (02/07/2011).

Original issue reported on code.google.com by ethan.sherrie on 2 Aug 2011 at 7:00

Attachments:

• PCTCompileExt_patches.txt

[GoogleCodeExporter]

Reproduced, thanks for your test case.
It seems that the procedure checking if r-code is already present or not is not 
working correctly.

I'll fix that ASAP and update this bug when a nightly build will contain the 
fix.

Original comment by g.quer...@gmail.com on 3 Aug 2011 at 9:58

• Changed state: Accepted
• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Do you have a timeline for this issue?

Thanks.

Original comment by ethan.bl...@sage.com on 12 Oct 2011 at 5:22

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

ASAP, but my main problem is the "P"... Not before two or three weeks.

I you want to implement the fix, the cause I've found when reproducing the bug 
lies in pct/pctBgCompile.p, line 263 to 267. Compilation units passed from Java 
to OpenEdge store output dir without package name (because OE preprend package 
names for us), and pctBgCompile doesn't take that into account.

The ASSIGN RCodeName = SUBSTRING(...) line should have a specific case for 
classes, and be able to deduce package name. 

Original comment by g.quer...@gmail.com on 12 Oct 2011 at 8:47

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Could you try build #152 ? Not ready for production use, there may be unknown 
bugs in PCTCompileExt as I changed the way compilation units are passed to the 
OpenEdge process. Test cases are OK though.

Original comment by g.quer...@gmail.com on 31 Oct 2011 at 10:20

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Assuming it's verified

Original comment by g.quer...@gmail.com on 8 Mar 2013 at 8:38

• Changed state: Fixed
• Added labels: ****
• Removed labels: ****