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

WIP: build: Update TizenRT to 2.0_Public_M2 #1780

Conversation

rzr
Copy link
Contributor

@rzr rzr commented Oct 22, 2018

Because current version of TizenRT in IoT.js docker image is oudated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

Use iotjs's defconfig as default config instead of file:
"tizenrt_release_config"
from container:
https://hub.docker.com/r/iotjs/ubuntu/

Is this file tracked anywhere?

Later debug, release configs can tracked in IoT.js's source tree,
and used as desired, for performance benchmarking.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgrade IoT.js ?)

Bug: #1777
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com

@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch 2 times, most recently from c0fbcb2 to 4f98356 Compare October 22, 2018 16:40
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from 4f98356 to 8f7bf9f Compare November 5, 2018 15:46
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 5, 2018
Because current version of TizenRT in IoT.js docker image is oudated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

Use iotjs's defconfig as default config instead of file:
"tizenrt_release_config"
from container:
https://hub.docker.com/r/iotjs/ubuntu/

Is this file tracked anywhere?

Later debug, release configs can tracked in IoT.js's source tree,
and used as desired, for performance benchmarking.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgrade IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from 8f7bf9f to f0744ce Compare November 5, 2018 16:29
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 5, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources.

File "config/tizenrt/release/artik05x/configs/defconfig"
is coming from "tizenrt_release_config" in travis build container:
https://hub.docker.com/r/iotjs/ubuntu/

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 5, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from f0744ce to cc60c36 Compare November 5, 2018 16:38
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 5, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources.

File "config/tizenrt/release/artik05x/configs/defconfig"
is coming from "tizenrt_release_config" in travis build container:
https://hub.docker.com/r/iotjs/ubuntu/

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 5, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from cc60c36 to e8a439c Compare November 5, 2018 17:11
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 5, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 7, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources.

File "config/tizenrt/release/artik05x/configs/defconfig"
is coming from "tizenrt_release_config" in travis build container:
https://hub.docker.com/r/iotjs/ubuntu/

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 8, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources' dir.

File "config/tizenrt/release/artik05x/configs/defconfig"
is coming from "tizenrt_release_config" in travis build container:

https://hub.docker.com/r/iotjs/ubuntu/

(I don't know if this file is tracked elsewhere ?)

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
Credit-to: iotjs <https://hub.docker.com/u/iotjs/>
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 8, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources' dir.

File "config/tizenrt/artik05x/configs/release/defconfig"
is coming from "tizenrt_release_config" in travis build container:

https://hub.docker.com/r/iotjs/ubuntu/

(I don't know if this file is tracked elsewhere ?)

Then debug file was aligned to "release" one,
with debug flags enabled.

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
Credit-to: iotjs <https://hub.docker.com/u/iotjs/>
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to rzr/iotjs that referenced this pull request Nov 8, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 8, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources' dir.

File "config/tizenrt/artik05x/configs/release/defconfig"
is coming from "tizenrt_release_config" in travis build container:

https://hub.docker.com/r/iotjs/ubuntu/

(I don't know if this file is tracked elsewhere ?)

Then debug file was aligned to "release" one,
with debug flags enabled.

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
Credit-to: iotjs <https://hub.docker.com/u/iotjs/>
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 8, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 8, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources' dir.

File "config/tizenrt/artik05x/configs/release/defconfig"
is coming from "tizenrt_release_config" in travis build container:

https://hub.docker.com/r/iotjs/ubuntu/

(I don't know if this file is tracked elsewhere ?)

Then debug file was aligned to "release" one,
with debug flags enabled.

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
Credit-to: iotjs <https://hub.docker.com/u/iotjs/>
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 8, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 8, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources' dir.

File "config/tizenrt/artik05x/configs/release/defconfig"
is coming from "tizenrt_release_config" in travis build container:

https://hub.docker.com/r/iotjs/ubuntu/

(I don't know if this file is tracked elsewhere ?)

Then debug file was aligned to "release" one,
with debug flags enabled.

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
Credit-to: iotjs <https://hub.docker.com/u/iotjs/>
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 8, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 9, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources.

File "config/tizenrt/release/artik05x/configs/defconfig"
is coming from "tizenrt_release_config" in travis build container:
https://hub.docker.com/r/iotjs/ubuntu/

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 9, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 9, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources' dir.

File "config/tizenrt/artik05x/configs/release/defconfig"
is coming from "tizenrt_release_config" in travis build container:

https://hub.docker.com/r/iotjs/ubuntu/

(I don't know if this file is tracked elsewhere ?)

Then debug file was aligned to "release" one,
with debug flags enabled.

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
Credit-to: iotjs <https://hub.docker.com/u/iotjs/>
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 9, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 9, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from e8a439c to e569f88 Compare November 9, 2018 09:53
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 9, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources.

File "config/tizenrt/release/artik05x/configs/defconfig"
is coming from "tizenrt_release_config" in travis build container:
https://hub.docker.com/r/iotjs/ubuntu/

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 9, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from e569f88 to d0c5faa Compare November 9, 2018 11:19
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 9, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr
Copy link
Contributor Author

rzr commented Nov 9, 2018

I am rebasing on upstream, I plan to get patches merged one per one:

1st in serie is:
#1792

yichoi pushed a commit that referenced this pull request Nov 12, 2018
Before updating TizenRT in build script,
I thought I would be better to maintain config files in sources' dir.

File "config/tizenrt/artik05x/configs/release/defconfig"
is coming from "tizenrt_release_config" in travis build container:

https://hub.docker.com/r/iotjs/ubuntu/

(I don't know if this file is tracked elsewhere ?)

Then debug file was aligned to "release" one,
with debug flags enabled.

Debug config has just been renamed, and script path adjusted.

Note for later, it would make sense to align both after upgrade.

Relate-to: #1780
Credit-to: iotjs <https://hub.docker.com/u/iotjs/>
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 12, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 12, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from d0c5faa to bca9381 Compare November 12, 2018 09:42
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 12, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 13, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 15, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
rzr added a commit to rzr/iotjs that referenced this pull request Nov 15, 2018
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from bca9381 to e9b9a5f Compare November 15, 2018 09:10
rzr added 2 commits November 20, 2018 15:45
Align to TizenRT's default config, but disable SSS for ARTIK053
also preserve debug options for debug configuration.

Bug: jerryscript-project#1777
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/tizenrt/review/master branch from e9b9a5f to 453931a Compare November 20, 2018 14:46
rzr added a commit to TizenTeam/iotjs that referenced this pull request Nov 20, 2018
Align to TizenRT's default config, but disable SSS for ARTIK053
also preserve debug options for debug configuration.

Because current version of TizenRT in IoT.js docker image is outdated,
let's fetch tag from upstream at build time,
to speed up build time, this part can be eventually removed
if image is updated with latest TizenRT's sources.

jerryx extra lib has been dropped for now,
this should be reintroduced later (once TizenRT upgraded IoT.js ?)

Bug: jerryscript-project#1777
Forwarded: jerryscript-project#1780
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr
Copy link
Contributor Author

rzr commented Nov 20, 2018

Or if you prefer the same changes in single squashed commit:
#1805

@rzr
Copy link
Contributor Author

rzr commented Nov 27, 2018

already in with squashed pr

@rzr rzr closed this Nov 27, 2018
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.

1 participant