Now that import paths are read from a file (r23593/r23593) in the Stratego editor, compatibility is broken because if the file doesn’t exist no import paths are used at all.

(Instead of the former hardcoded paths…)

EDIT:

[15:39]	<tvo>	oh there are some defaults in the code
[15:39]	<tvo>	maybe something to do with absolute vs relative imports then

[15:49]	<karltk>	the paths may include double-slashes, but I though that would work just fine everywhere
[15:53]	<tvo>	karltk: I suppose expand-relative-path is to blame, on windows rel path can't be detected using  not(?'/')
[15:54]	<karltk>	tvo: ah, right!
[15:55]	<karltk>	we should rather use is-relpath and then also fix is-abspath to work for windows
[16:02]	<Chip_Zero>	wasn't is-abspath fixed already?
[16:05]	<Chip_Zero>	oh, but it wasn't used here
Submitted by Tobi Vollebregt on 21 November 2011 at 15:29

On 21 November 2011 at 15:29 Tobi Vollebregt tagged stratego

On 21 November 2011 at 15:29 Tobi Vollebregt tagged incompatible

On 25 November 2011 at 16:18 Tobi Vollebregt commented:

This should be solved in SVN, except that spoofax-imp-trunk uses an old libstratego-lib, with an is-abspath that doesn’t understand windows paths with forward slashes (which Eclipse and/or the Spoofax runtime pass to Stratego).

I’ve already forced rebuild of stratego-libraries, spoofax-imp-trunk, strategoxt-java in hydra (the last one seemed to automatically trigger another spoofax-imp-trunk rebuild), but this didn’t help, the nightly update site still ships with the old libraries.

Now I’ve no clue what else / in which order to rebuild things, and I don’t want to spam SVN too much with ‘push’ commits, so would be nice if someone else can fix it / tell what step I’m missing :-)


On 28 November 2011 at 14:57 Lennart Kats removed tag incompatible

On 28 November 2011 at 14:57 Lennart Kats tagged interesting

On 28 November 2011 at 14:57 Lennart Kats tagged 1.0

On 29 November 2011 at 11:45 Tobi Vollebregt commented:

This is solved now, thanks!


On 29 November 2011 at 11:45 Tobi Vollebregt closed this issue.

On 29 November 2011 at 11:54 Lennart Kats removed tag interesting

On 28 December 2011 at 13:11 Lennart Kats tagged @tvo

On 29 February 2012 at 17:30 Karl Trygve Kalleberg tagged karltk

On 29 February 2012 at 17:30 Karl Trygve Kalleberg removed tag karltk

Log in to post comments