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

Make functionbeat build process depends on linux/amd64 #8889

Merged

Conversation

ph
Copy link
Contributor

@ph ph commented Nov 1, 2018

Functionbeat packaging depends on a Linux binary that will be sent to
the serverless platform, previously the magefile for the project did not express
that dependency. This was causing problems when you were building the
packages only for a specific platform. This commit fixes that problem by
introducing mage.WithPlatforms this allows you to specific dependency
when you define the cross build logic.

@ph ph added :Packaging needs_backport PR is waiting to be backported to other branches. Functionbeat labels Nov 1, 2018
@ph ph requested a review from andrewkroh November 1, 2018 17:52
Functionbeat packaging depends on a linux binary that will be send to
the serverless platform, previously the magefile for the project did not express
that dependencies. This was causing problem when you were building the
packages only for a specific platform. This commit fixes that problem by
introducing `mage.WithPlatforms` this allow you to specific dependency
when you define the cross build logic.
@ph ph force-pushed the fix/allow-to-define-dependency-on-other-platforms branch from 82c6de1 to 61d3cc3 Compare November 1, 2018 17:53
@ph ph added the review label Nov 1, 2018
Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

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

Nice solution.

dev-tools/mage/crossbuild.go Outdated Show resolved Hide resolved
dev-tools/mage/platforms.go Show resolved Hide resolved
@ph
Copy link
Contributor Author

ph commented Nov 1, 2018

@andrewkroh Update with your changes.

@ph ph merged commit e2d448e into elastic:master Nov 2, 2018
ph added a commit to ph/beats that referenced this pull request Nov 2, 2018
Functionbeat packaging depends on a Linux binary that will be sent to
the serverless platform, previously the magefile for the project did not express
that dependency. This was causing problems when you were building the
packages only for a specific platform. This commit fixes that problem by
introducing `mage.WithPlatforms` this allows you to specific dependency
when you define the cross build logic.

(cherry picked from commit e2d448e)
@ph ph added v6.5.0 and removed needs_backport PR is waiting to be backported to other branches. labels Nov 2, 2018
ph added a commit to ph/beats that referenced this pull request Nov 2, 2018
Functionbeat packaging depends on a Linux binary that will be sent to
the serverless platform, previously the magefile for the project did not express
that dependency. This was causing problems when you were building the
packages only for a specific platform. This commit fixes that problem by
introducing `mage.WithPlatforms` this allows you to specific dependency
when you define the cross build logic.

(cherry picked from commit e2d448e)
@ph ph added the v6.6.0 label Nov 2, 2018
@jsoriano
Copy link
Member

jsoriano commented Nov 8, 2018

Should we set the platforms list to just linux/amd64 instead of adding it? Does it make sense to build functionbeat for non-linux platforms?

@ph
Copy link
Contributor Author

ph commented Nov 8, 2018

@jsoriano it is required, since the CLI can be run on linux, windows and mac but the binary on the actual serverless provider is linux/amd64.

@jsoriano
Copy link
Member

jsoriano commented Nov 8, 2018

Oh ok, I see, I was not thinking on the CLI, thanks!

ph added a commit that referenced this pull request Nov 8, 2018
…linux/amd64 (#8904)

Cherry-pick of PR #8889 to 6.x branch. Original message: 

Functionbeat packaging depends on a Linux binary that will be sent to
the serverless platform, previously the magefile for the project did not express
that dependency. This was causing problems when you were building the
packages only for a specific platform. This commit fixes that problem by
introducing `mage.WithPlatforms` this allows you to specific dependency
when you define the cross build logic.
ph added a commit that referenced this pull request Nov 15, 2018
Functionbeat packaging depends on a Linux binary that will be sent to
the serverless platform, previously the magefile for the project did not express
that dependency. This was causing problems when you were building the
packages only for a specific platform. This commit fixes that problem by
introducing `mage.WithPlatforms` this allows you to specific dependency
when you define the cross build logic.

(cherry picked from commit e2d448e)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…elastic#8903)

Functionbeat packaging depends on a Linux binary that will be sent to
the serverless platform, previously the magefile for the project did not express
that dependency. This was causing problems when you were building the
packages only for a specific platform. This commit fixes that problem by
introducing `mage.WithPlatforms` this allows you to specific dependency
when you define the cross build logic.

(cherry picked from commit f71fc5c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants