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

Work around error building deb on Windows #47011

Merged
merged 4 commits into from
Sep 27, 2019

Conversation

alpar-t
Copy link
Contributor

@alpar-t alpar-t commented Sep 24, 2019

Relates to #47007 . the gradle-ospackage-plugin plugin doesn't
properly support symlink on windows.

This PR changes the way we configure tasks to prevent building these
packages as part of a windows check.

Relates to  elastic#47007 . the `gradle-ospackage-plugin` plugin doesn't
properly support symlink on windows.

This PR changes the way we configure tasks to prevent building these
packages as part of a windows check.
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@alpar-t alpar-t added the :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts label Sep 24, 2019
@alpar-t
Copy link
Contributor Author

alpar-t commented Sep 24, 2019

@elasticmachine run elasticsearch-ci/packaging

Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor comment.

distribution/packages/build.gradle Outdated Show resolved Hide resolved
Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yikes, what happened, did the whole file get reformatted?

@alpar-t
Copy link
Contributor Author

alpar-t commented Sep 25, 2019

Yikes, what happened, did the whole file get reformatted?

@jasontedor viewing the diff with white space changes ignored shows a much better picture. All the lines got an additional indent following the suggestion made by Mark to avoid the return.

@alpar-t
Copy link
Contributor Author

alpar-t commented Sep 25, 2019

@elasticmachine run elasticsearch-ci/2

@jasontedor
Copy link
Member

Thanks for that @atorok.

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@alpar-t
Copy link
Contributor Author

alpar-t commented Sep 25, 2019

@elasticmachine run elasticsearch-ci/packaging There's a known issue, but this doesn't seem to be it.

@alpar-t
Copy link
Contributor Author

alpar-t commented Sep 26, 2019

It looks like it takes more work to go to the latest version of the os distribution plugin. Reverting for now as it's not strictly needed for this change

@alpar-t
Copy link
Contributor Author

alpar-t commented Sep 27, 2019

This time the failure is the one from #46903

@alpar-t alpar-t merged commit e66eeca into elastic:master Sep 27, 2019
alpar-t added a commit that referenced this pull request Sep 27, 2019
Relates to  #47007 . the `gradle-ospackage-plugin` plugin doesn't
properly support symlink on windows.

This PR changes the way we configure tasks to prevent building these
packages as part of a windows check.
alpar-t added a commit that referenced this pull request Sep 27, 2019
Relates to  #47007 . the `gradle-ospackage-plugin` plugin doesn't
properly support symlink on windows.

This PR changes the way we configure tasks to prevent building these
packages as part of a windows check.
@colings86 colings86 added v7.4.0 and removed v7.4.1 labels Sep 27, 2019
@alpar-t alpar-t deleted the fix-deb-windows branch November 11, 2019 09:37
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Build Build or test infrastructure :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts Team:Delivery Meta label for Delivery team v7.4.0 v7.5.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants