Incompatibility with rollup.js starting with 3.0.6 - Part 2 #2224

Closed
btakita opened this Issue Jan 15, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@btakita

btakita commented Jan 15, 2017

This is an open issue (for tracking purposes) to #2212.

I created this issue because #2212 is closed yet the bug still remains...

There is a fix at #2222

Thank you for developing riot. I need this bug fixed, as this causes an exception.

@btakita

This comment has been minimized.

Show comment
Hide comment
@btakita

btakita Jan 15, 2017

Here's the demonstration. Why don't you believe me that this is a bug?

https://github.com/btakita/riot-rollup-server-error

btakita commented Jan 15, 2017

Here's the demonstration. Why don't you believe me that this is a bug?

https://github.com/btakita/riot-rollup-server-error

@btakita

This comment has been minimized.

Show comment
Hide comment
@btakita

btakita Jan 15, 2017

Here's the output. Note the _interopDefault function provided by rollup

https://github.com/btakita/riot-rollup-server-error/blob/master/dist/cjs.js

$ ./build.sh 

/home/phoenix/workspace/riot/riot-rollup-server-error/dist/cjs.js:7
if (!riot.render) throw 'riot.render does not exist'
                  ^
riot.render does not exist

btakita commented Jan 15, 2017

Here's the output. Note the _interopDefault function provided by rollup

https://github.com/btakita/riot-rollup-server-error/blob/master/dist/cjs.js

$ ./build.sh 

/home/phoenix/workspace/riot/riot-rollup-server-error/dist/cjs.js:7
if (!riot.render) throw 'riot.render does not exist'
                  ^
riot.render does not exist
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment