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

cmake: Allow to select python3 from env #1946

Conversation

rzr
Copy link
Contributor

@rzr rzr commented Jul 24, 2020

This will be useful for next debian release shipping python3

Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
Bug: #1945
Bug-Debian: https://bugs.debian.org/936738
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]

rzr added a commit to TizenTeam/iotjs that referenced this pull request Jul 24, 2020
This will be useful for next debian release shipping python3

Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]
@rzr rzr force-pushed the sandbox/rzr/python/review/master branch from 096f414 to 0a3af42 Compare July 24, 2020 11:32
Copy link
Member

@zherczeg zherczeg left a comment

Choose a reason for hiding this comment

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

LGTM

@akosthekiss
Copy link
Member

Travis job is pretty much red: https://travis-ci.org/github/jerryscript-project/iotjs/builds/711429162 I think it should be fixed first. (I'm not sure why the job status is not shown here at the PR.)

rzr added a commit to TizenTeam/iotjs that referenced this pull request Jul 24, 2020
This will be useful for next debian release shipping python3

Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]
@rzr
Copy link
Contributor Author

rzr commented Jul 24, 2020

It looks like build is not stable, for instance this trivial change did not break:

https://travis-ci.org/github/jerryscript-project/iotjs/builds/711220082

I'll rebase harmless patches.

rzr added a commit to TizenTeam/iotjs that referenced this pull request Jul 25, 2020
This will be useful for next debian release shipping python3

Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]

Change-Id: Id4509a97f05eb77a40145b7aa50360b92f806fdf
@rzr rzr force-pushed the sandbox/rzr/python/review/master branch from 0a3af42 to d005559 Compare July 25, 2020 05:10
rzr added a commit to TizenTeam/iotjs that referenced this pull request Jul 25, 2020
This will be useful for next debian release shipping python3

Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
Last-Update: 2020-07-25
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]

Change-Id: I48bf135d7a3af6e64859252078931ff3388a87a6
rzr added a commit to TizenTeam/iotjs that referenced this pull request Jul 25, 2020
This will be useful for next debian release shipping python3

Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]

Change-Id: Ic0db03346959f2760439aac84a0cbc166fc54d0d
@rzr rzr force-pushed the sandbox/rzr/python/review/master branch from d005559 to b2b0cfe Compare July 25, 2020 08:49
rzr added a commit to TizenTeam/iotjs that referenced this pull request Jul 27, 2020
This will be useful for next debian release shipping python3

Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]
@rzr rzr force-pushed the sandbox/rzr/python/review/master branch from b2b0cfe to 574a545 Compare July 27, 2020 07:54
Copy link
Contributor

@galpeter galpeter left a comment

Choose a reason for hiding this comment

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

In this PR the deps/jerry is also updated. I don't think we need that in this PR.

rzr added a commit to TizenTeam/iotjs that referenced this pull request Jul 27, 2020
This will be useful for next debian release shipping python3

Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]
@rzr rzr force-pushed the sandbox/rzr/python/review/master branch from 574a545 to 3e75379 Compare July 27, 2020 20:25
@rzr
Copy link
Contributor Author

rzr commented Jul 27, 2020

thx for finding this mistake

This will be useful for next debian release shipping python3

Version can be specified as cmake option using:

    CMAKE_OPTIONS=-DPYTHON="python3"

Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
Relate-to: https://travis-ci.org/github/TizenTeam/iotjs/builds/712336077
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]
@rzr rzr force-pushed the sandbox/rzr/python/review/master branch from 3e75379 to 91a75a2 Compare July 28, 2020 08:35
@rzr
Copy link
Contributor Author

rzr commented Jul 28, 2020

I suspect a travis issue because it built my side:

https://travis-ci.org/github/TizenTeam/iotjs/builds/712336077

rzr added a commit to rzr/iotjs that referenced this pull request Jul 29, 2020
This will be useful for next debian release shipping python3

Change-Id: Ie3a9e8ea9152247efa85a4e87bd121016a466795
Bug: jerryscript-project#1945
Bug-Debian: https://bugs.debian.org/936738
Forwarded: jerryscript-project#1946
Origin: https://github.com/TizenTeam/iotjs/tree/sandbox/rzr/python/review/master
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval [email protected]
Copy link
Contributor

@galpeter galpeter left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@akosthekiss akosthekiss left a comment

Choose a reason for hiding this comment

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

LGTM

@akosthekiss akosthekiss merged commit bf715a9 into jerryscript-project:master Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants