Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maven vaadin:dance fails most of the time #13760

Closed
simasch opened this issue May 13, 2022 · 4 comments
Closed

Maven vaadin:dance fails most of the time #13760

simasch opened this issue May 13, 2022 · 4 comments

Comments

@simasch
Copy link

simasch commented May 13, 2022

Description of the bug

After migrating to Vaadin 23 the dance fails 80% of the time and I have to delete the node_modules folder manually.

The error message is

[ERROR] Failed to execute goal com.vaadin:vaadin-maven-plugin:23.0.6:dance (dance) on project tosca-develop: Execution dance of goal com.vaadin:vaadin-maven-plugin:23.0.6:dance failed: java.nio.file.NoSuchFileException: C:\Users\simon\Workspace\Dynasoft\tosca-ui\tosca-develop\node_modules@vaadin\flow-frontend -> [Help 1]

Expected behavior

Dance should delete the generated files.

Minimal reproducible example

Create a project and run it on Windows.

Versions

  • Vaadin / Flow version: 23.x
  • Java version: 17
  • OS version: Windows (10 or 11)
@Artur-
Copy link
Member

Artur- commented May 13, 2022

Is there always a missing \ between node_modules and@?

@caalador
Copy link
Contributor

Windows has issues with symbolic links.
For instance folders are not recognized as symbolic links and deletion tries to remove the actual folder.

This will generate this kind of exception always if you do mvn clean vaadin:clean-frontend, but will not happen if you do mvn vaadin:clean-frontend clean as then the target file is still available for the symlink.

@simasch
Copy link
Author

simasch commented May 13, 2022

@Artur- No. That's probably a typo.

[ERROR] Failed to execute goal com.vaadin:vaadin-maven-plugin:23.0.6:dance (dance) on project tosca-develop: Execution dance of goal com.vaadin:vaadin-maven-plugin:23.0.6:dance failed: java.nio.file.NoSuchFileException: C:\Users\simon\Workspace\Dynasoft\tosca-ui\tosca-develop\node_modules\@vaadin\flow-frontend -> [Help 1]

@simasch
Copy link
Author

simasch commented May 13, 2022

@caalador I understand thanks!

So then I'll to do dance before clean for example

<execution>
<id>dance</id>
  <phase>pre-clean</phase>
  <goals>
    <goal>dance</goal>
  </goals>
</execution>

@simasch simasch closed this as completed May 13, 2022
caalador added a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760
caalador added a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760
mshabarov pushed a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760
mshabarov pushed a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760
vaadin-bot pushed a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760
vaadin-bot pushed a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760
vaadin-bot pushed a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760
vaadin-bot pushed a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760
vaadin-bot added a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760

Co-authored-by: caalador <[email protected]>
vaadin-bot added a commit that referenced this issue May 13, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760

Co-authored-by: caalador <[email protected]>
vaadin-bot added a commit that referenced this issue May 13, 2022
… (#13774)

Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760

Co-authored-by: caalador <[email protected]>
mshabarov pushed a commit that referenced this issue May 16, 2022
Move clean-frontend to pre-clean as if
mvn clean is executed before clean-frontend
clean will remove the target folder making
removal of node_modules fail on windows
if there is a symlink to target/flow-frontend

touches #13760

Co-authored-by: caalador <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants