conda-forge / or-tools-feedstock

A conda-smithy repository for or-tools.
BSD 3-Clause "New" or "Revised" License
2 stars 8 forks source link

Use master as source [DO NOT MERGE] #15

Closed BastianZim closed 1 year ago

BastianZim commented 2 years ago

Checklist

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

BastianZim commented 2 years ago

@conda-forge-admin, please rerender

github-actions[bot] commented 2 years ago

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/or-tools-feedstock/actions/runs/2162328575.

BastianZim commented 2 years ago

Hi @Mizux This would be the way to test whatever you have on master. You can just restart the PR by adding the following comment to this PR: @conda-forge-admin, please restart ci.

I didn't update the deps yet as I wasn't sure what they are going to be. Can you post any change here and I'll update it?

I can also give you some instructions on how you can do that yourself but this seemed easier – let me know if you still want them.

Mizux commented 2 years ago

@BastianZim can you test with this diff ?

%git diff -u origin/main 
diff --git a/recipe/meta.yaml b/recipe/meta.yaml
index 1d790cb..fae5041 100644
--- a/recipe/meta.yaml
+++ b/recipe/meta.yaml
@@ -1,10 +1,10 @@
 {% set name = "or-tools" %}
 {% set package_name = "ortools" %}
-{% set version = "9.2" %}
+{% set version = "9.4" %}
 {% set home = "https://developers.google.com/optimization" %}
 {% set license = "Apache-2.0" %}
 {% set license_family = "Apache" %}
-{% set license_file = "LICENSE-2.0.txt" %}
+{% set license_file = "LICENSE" %}
 {% set dev_url = "https://github.com/google/or-tools" %}

@@ -13,13 +13,13 @@ package:
   version: {{ version }}

 source:
-  url: https://github.com/google/or-tools/archive/v{{ version }}.tar.gz
-  sha256: 5337935ea1fa010bb62cf0fc8bedd6de07dda77bff3db7a0f6a36c84c7bd58db
+  git_url: https://github.com/google/or-tools.git
+  git_tag: master

 # The meta package

 build:
-  number: 3
+  number: 0
   skip: true  # [not linux]

 outputs:
@@ -39,6 +39,7 @@ outputs:
       host:
         - abseil-cpp
         - libprotobuf
+        - re2
         - zlib 1.2.11
         # coin or deps
         - coin-or-utils

note: you can also reset hard your [test-master] branch on my branch main

something along this line...

git remote add mizux git@github.com:Mizux/or-tools-feedstock.git
git fetch --all
git checkout test-master
git reset --hard mizux/main
git push -f

see: https://github.com/Mizux/or-tools-feedstock/commits/main

note: re2 seems to be available, so I would like to test if or-tools correctly manage to find it ;) ref: https://github.com/conda-forge/re2-feedstock

BastianZim commented 2 years ago

@conda-forge-admin, please rerender

BastianZim commented 2 years ago

Have added the dif, let's see if it works :)

note: you can also reset hard your [test-master] branch on my branch main

I can do that but it might be easier if you just fork this repo and apply it yourself and make a PR, then you don't need to wait for me? Whichever is easier for you. The CI runs on every PR so you can just try it. You just need to make sure to post the rerender comment above so https://github.com/conda-forge/or-tools-feedstock/pull/15#issuecomment-1102721693 that the rest of the infrastructure gets updated accordingly, if you change the deps etc. Basically, the meta.yaml file is the instruction for how to build the package and the actual steps are in the other files which will sometimes get updated when you rerender.

re2 seems to be available, so I would like to test if or-tools correctly manage to find it ;)

Also added it :)