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

[FLINK-5814] fix packaging flink-dist in unclean source directory #3331

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@NicoK
Contributor

NicoK commented Feb 16, 2017

If <flink-dir>/build-target already existed, running mvn package for
flink-dist would create a symbolic link inside <flink-dir>/build-target
instead of replacing that symlink. This commit fixes this behaviour of ln -sf
by adding the --no-dereference parameter.

[FLINK-5814] fix packaging flink-dist in unclean source directory
If "<flink-dir>/build-target" already existed, running 'mvn package' for
flink-dist would create a symbolic link inside "<flink-dir>/build-target"
instead of replacing that symlink. This commit fixes this behaviour of 'ln -sf'
by adding the --no-dereference parameter.
@StephanEwen

This comment has been minimized.

Show comment
Hide comment
@StephanEwen

StephanEwen Feb 16, 2017

Contributor

+1 merging this...

Contributor

StephanEwen commented Feb 16, 2017

+1 merging this...

@asfgit asfgit closed this in 2ec2abf Feb 16, 2017

StephanEwen added a commit to StephanEwen/incubator-flink that referenced this pull request Feb 16, 2017

[FLINK-5814] [build] Fix packaging flink-dist in unclean source direc…
…tory

If "<flink-dir>/build-target" already existed, running 'mvn package' for
flink-dist would create a symbolic link inside "<flink-dir>/build-target"
instead of replacing that symlink. This commit fixes this behaviour of 'ln -sf'
by adding the --no-dereference parameter.

This closes apache#3331
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment