embroider-build / embroider

Compiling Ember apps into spec-compliant, modern Javascript.
MIT License
337 stars 136 forks source link

Can't resolve ./-computed-store in ...ember-macro-helpers #285

Closed ChristopherConnock closed 2 years ago

ChristopherConnock commented 5 years ago

Building on Windows 10 with Ember 3.11.

I am getting the following error with ember-macro-helpers. I tried forcing the use of the latest build (4.2.0) to no avail. It is used in ember-moment and ember-awesome-macros.

ERROR Summary:

=================================================================================

scottkidder commented 5 years ago

I can confirm this on Linux on ember-cli@3.10.1 as well.

It seems to be caused by ember-macro-helpers not having a file at ./-computed-store and instead providing -computed-store by doing a monkey patch here

This change was introduced recently.

Is there configuration in embroider to ignore or workaround this type of situation?

void-mAlex commented 4 years ago

Ran into this as well running ember-cli@3.15.2 linux Any workarounds?

ef4 commented 4 years ago

I think this is fixed on master.

On Fri, Feb 7, 2020 at 6:57 PM Alex notifications@github.com wrote:

Ran into this as well running ember-cli@3.15.2 linux Any workarounds?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/embroider-build/embroider/issues/285?email_source=notifications&email_token=AACN6MXO3CEX7EMVKLFLLXTRBXYO3A5CNFSM4IEINJK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELFAPSA#issuecomment-583665608, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACN6MW4CJ4PQWCS2QMJDFTRBXYO3ANCNFSM4IEINJKQ .

void-mAlex commented 4 years ago

thank you for the quick response I've followed to CONTRIBUTING steps to compile and try out master and I can confirm the build now completes and assets get generated but the whole app no longer functions I've seen these warnings for the build:

WARNING in ./node_modules/ember-data/adapters/errors.js 1:0-213
"export 'AdapterError' was not found in '@ember-data/adapter/error'
 @ ./assets/blog.js

WARNING in ./helpers/not-eq.js 1:0-87
"export 'notEq' was not found in '../node_modules/ember-truth-helpers/helpers/not-equal'
 @ ./assets/blog.js

WARNING in ./helpers/responsive-image-resolve.js 1:0-122
"export 'responsiveImageResolve' was not found in '../node_modules/ember-responsive-image/helpers/responsive-image-resolve'
 @ ./assets/blog.js

WARNING in ./helpers/route-is.js 1:0-81
"export 'routeIs' was not found in '../node_modules/empress-blog/helpers/route-is'
 @ ./assets/blog.js

WARNING in ./helpers/utc.js 1:0-114
"export 'utc' was not found in '../node_modules/empress-blog-casper-template/node_modules/ember-moment/helpers/utc'
 @ ./assets/blog.js

should I raise this as a separate issue? for some other quick info that might be relevant it's a blog app using empress prember and a default template

ef4 commented 4 years ago

This warnings are generally harmless but they’re all real bugs in the addons they’re talking about. These addons try to reexport nonexistent things.

It would be helpful for someone to send PRs that clean these up so people stop seeing the warnings.

On Sat, Feb 8, 2020 at 8:37 AM Alex notifications@github.com wrote:

thank you for the quick response I've followed to CONTRIBUTING steps to compile and try out master and I can confirm the build now completes and assets get generated but the whole app no longer functions I've seen these warnings for the build:

WARNING in ./node_modules/ember-data/adapters/errors.js 1:0-213 "export 'AdapterError' was not found in '@ember-data/adapter/error' @ ./assets/blog.js

WARNING in ./helpers/not-eq.js 1:0-87 "export 'notEq' was not found in '../node_modules/ember-truth-helpers/helpers/not-equal' @ ./assets/blog.js

WARNING in ./helpers/responsive-image-resolve.js 1:0-122 "export 'responsiveImageResolve' was not found in '../node_modules/ember-responsive-image/helpers/responsive-image-resolve' @ ./assets/blog.js

WARNING in ./helpers/route-is.js 1:0-81 "export 'routeIs' was not found in '../node_modules/empress-blog/helpers/route-is' @ ./assets/blog.js

WARNING in ./helpers/utc.js 1:0-114 "export 'utc' was not found in '../node_modules/empress-blog-casper-template/node_modules/ember-moment/helpers/utc' @ ./assets/blog.js

should I raise this as a separate issue? for some other quick info that might be relevant it's a blog app using empress prember and a default template

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/embroider-build/embroider/issues/285?email_source=notifications&email_token=AACN6MQS7F5FEFICCYP2AZ3RB2YTDA5CNFSM4IEINJK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELFR76A#issuecomment-583737336, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACN6MXK7K4GNMWJ33ZBKM3RB2YTDANCNFSM4IEINJKQ .

void-mAlex commented 4 years ago

381 fixed this for me

Windvis commented 2 years ago

Closing since the original issue seems to be fixed.