svg in tag breaks in 2.3.21 about compiler HOT 13 CLOSED

@justinmchase , the output that I get seems correct, can you check it please?:

//src: test/specs/fixtures/svg.tag
riot.tag2('icon-heart',
  '<svg viewbox="0 0 32 32" version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink"><path d="M16,28.261c0,0-14-7.926-14-17.046c0-9.356,13.159-10.399,14-0.454c1.011-9.938,14-8.903,14,0.454 C30,20.335,16,28.261,16,28.261z"></path> </svg>', '', '', function(opts) {
});

from compiler.

When I run it like that I am not seeing a problem either. We are specifically using gulp-riot in our build to process the tags.

We are specifically using [email protected] and [email protected].

I'm trying to find a more targeted repro for you.

from compiler.

I've attached a minimal reproduction of the issue.
repro-45.zip

Unzip then:

$ cd test
$ npm install
$ gulp good # success
$ gulp bad # hangs

You'll see that the good.tag processes without error, as expected, but bad.tag when run through the same pipeline causes gulp to hang, no error is produced, and no tag file is generated.

from compiler.

@justinmchase , this is a bug. Please use a previous riot version, I will fix this issue ASAP.

from compiler.

For anyone encountering this issue the workaround was to add "riot-compiler": "2.3.20" to my main package.json which causes it to use that version specifically.

from compiler.

Had this issue too, oof that was a hard one to track down...

To provide some content, this was causing webpack to hang with riotjs-loader, I have a tag with a svg in it as well.

from compiler.

The error is in a regex parsing quoted attribute values.

from compiler.

Please test with v2.3.22, it is already in npm.

from compiler.

Appears to work now, thanks @aMarCruz !!!

from compiler.

Works for me too. I'm closing this for now as fixed by #44

from compiler.

So unfortunately I had to revert back to 2.3.20 again because, while this seems to work at first it is still hanging our build. It does successfully generate the files and the compiler seems to work but it then hangs open our gulp process and it cannot be killed with Ctrl+c.

Revertting back to the older version fixes the problem. I'm not totally sure but it seems like a callback may not be called in this situation.

from compiler.

Shoot false alarm, I think it's something else taht was masked by this problem after all. Sorry.

from compiler.

good to hear it.

from compiler.