Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

APM is unable to install with scoped packages #702

Closed
1 task done
TimvdLippe opened this issue Mar 16, 2017 · 3 comments · Fixed by #705
Closed
1 task done

APM is unable to install with scoped packages #702

TimvdLippe opened this issue Mar 16, 2017 · 3 comments · Fixed by #705
Labels

Comments

@TimvdLippe
Copy link
Contributor

Prerequisites

Description

We are using the build script provided by Atom (https://raw.githubusercontent.com/atom/ci/master/build-package.sh) which should be installing our packages. Since our Atom package is built with Typescript, we have to first compile the code before we can start the actual tests. This requires installation of our packages with NPM/Yarn.

If APM is then invoked to install packages, it gets into an endless loop trying to remove and install scoped packages: https://travis-ci.org/Polymer/atom-plugin/builds/211494054#L234

Steps to Reproduce

  1. Run npm install on an Atom package with a scoped package (for example @types/atom)
  2. Use the build script
  3. Run on Travis, but reproduces locally too

Expected behavior: Installation continues

Actual behavior: Endless loop trying to remove scoped packages during installation

Reproduces how often: 100%

Versions

You can get this information from copy and pasting the output of atom --version and apm --version from the command line. Also, please include the OS and what version of the OS you're running.

Downloading latest Atom release...
Using Atom version:
Atom    : 1.16.0-beta0
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0
Using APM version:
apm  1.16.1
npm  3.10.5
node 6.9.5 x64
python 2.7.3
git 1.8.5.6

Additional Information

Any additional information, configuration or data that might be necessary to reproduce the issue.

@43081j
Copy link

43081j commented Mar 17, 2017

FYI this can also be reproduced IIRC by:

npm install
apm clean

the clean call gets stuck in loops trying to clean scoped packages.

@TimvdLippe
Copy link
Contributor Author

@43081j good call, it indeed was the clean command. I submitted a fix in #705

@winstliu winstliu added the bug label Mar 18, 2017
@Alhadis
Copy link

Alhadis commented Mar 21, 2017

I wish I could physically punch apm.

Alhadis added a commit to Alhadis/Utils that referenced this issue Mar 21, 2017
Alhadis added a commit to file-icons/atom-fs that referenced this issue Mar 21, 2017
Alhadis added a commit to file-icons/atom that referenced this issue Mar 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants