Skip to content
This repository has been archived by the owner on Jun 26, 2024. It is now read-only.

Add install-tools and local-env rules to Makefiles #1280

Merged

Conversation

filariow
Copy link
Contributor

If required tool version is found on the host, then it will be linked into the ./bin folder. Otherwise, it will be downloaded.

Signed-off-by: Francesco Ilario [email protected]

Changes

Added install-tools and local-env rules to makefiles.
Leveraging on these rules, it is now possible to easily install the needed version of each dependency.

/kind enhancement

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

  • Docs
    included if any changes are user facing
  • Tests
    included if any functionality added or changed. For bugfixes please include tests that can catch regressions
  • All acceptance test scenarios included in the PR which verifies a bugfix or a requested feature reported by a non-member are tagged with @external-feedback tag.
  • Follows the commit message standard

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 19, 2022

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot added the kind/enhancement New feature or request label Oct 19, 2022
@codecov
Copy link

codecov bot commented Oct 19, 2022

Codecov Report

Merging #1280 (9d47f28) into master (ba95a74) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1280   +/-   ##
=======================================
  Coverage   58.06%   58.06%           
=======================================
  Files          33       33           
  Lines        2797     2797           
=======================================
  Hits         1624     1624           
  Misses       1018     1018           
  Partials      155      155           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba95a74...9d47f28. Read the comment docs.

@filariow filariow marked this pull request as ready for review October 19, 2022 15:05
@filariow filariow force-pushed the pr-install-tools branch 5 times, most recently from b9a3080 to cb11828 Compare October 19, 2022 21:40
@pmacik pmacik added release/v1.2.x Used to mark PRs to be cherry-picked in release-v1.2.x branch release/v1.3.x Used to mark PRs to be cherry-picked in release-v1.3.x branch labels Oct 24, 2022
make/common.mk Outdated Show resolved Hide resolved
If required tool version is found on the host, then it will be linked
into the ./bin folder. Otherwise, it will be downloaded.

Signed-off-by: Francesco Ilario <[email protected]>
@pmacik
Copy link
Contributor

pmacik commented Oct 24, 2022

/lgtm
/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 24, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pmacik

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot merged commit b983cfc into redhat-developer:master Oct 24, 2022
service-binding-operator-bot pushed a commit to service-binding-operator-bot/service-binding-operator that referenced this pull request Oct 24, 2022
…1280)

If required tool version is found on the host, then it will be linked
into the ./bin folder. Otherwise, it will be downloaded.

Signed-off-by: Francesco Ilario <[email protected]>

Signed-off-by: Francesco Ilario <[email protected]>
service-binding-operator-bot pushed a commit to service-binding-operator-bot/service-binding-operator that referenced this pull request Oct 24, 2022
…1280)

If required tool version is found on the host, then it will be linked
into the ./bin folder. Otherwise, it will be downloaded.

Signed-off-by: Francesco Ilario <[email protected]>

Signed-off-by: Francesco Ilario <[email protected]>
@filariow filariow deleted the pr-install-tools branch October 24, 2022 12:37
openshift-merge-robot pushed a commit that referenced this pull request Oct 24, 2022
If required tool version is found on the host, then it will be linked
into the ./bin folder. Otherwise, it will be downloaded.

Signed-off-by: Francesco Ilario <[email protected]>

Signed-off-by: Francesco Ilario <[email protected]>

Signed-off-by: Francesco Ilario <[email protected]>
Co-authored-by: Francesco Ilario <[email protected]>
pmacik pushed a commit to service-binding-operator-bot/service-binding-operator that referenced this pull request Oct 25, 2022
…1280)

If required tool version is found on the host, then it will be linked
into the ./bin folder. Otherwise, it will be downloaded.

Signed-off-by: Francesco Ilario <[email protected]>

Signed-off-by: Francesco Ilario <[email protected]>
openshift-merge-robot pushed a commit that referenced this pull request Oct 25, 2022
If required tool version is found on the host, then it will be linked
into the ./bin folder. Otherwise, it will be downloaded.

Signed-off-by: Francesco Ilario <[email protected]>

Signed-off-by: Francesco Ilario <[email protected]>

Signed-off-by: Francesco Ilario <[email protected]>
Co-authored-by: Francesco Ilario <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved kind/enhancement New feature or request lgtm release/v1.2.x Used to mark PRs to be cherry-picked in release-v1.2.x branch release/v1.3.x Used to mark PRs to be cherry-picked in release-v1.3.x branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants