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.

Sign up for GitHub

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

Jump to bottom

[infra] Remove BUILD_STAMP #8598

Closed
hseok-oh opened this issue Mar 8, 2022 · 2 comments
Closed

[infra] Remove BUILD_STAMP #8598

hseok-oh opened this issue Mar 8, 2022 · 2 comments
Labels
area/CI For our Continuous Integration

Comments

@hseok-oh
Copy link
Contributor

hseok-oh commented Mar 8, 2022 •
edited
Loading

It's better to remove BUILD_STAMP then maintain INSTALL_STAMP only. It was introduced to reduce build overhead on CI when external library build failed. But its side-effect makes difficult to resolve issue when

  • External library version is updated
  • Build fail is resolved on local

Originally posted by @hseok-oh in #8586 (comment)

Also related with #8487
@hseok-oh
Copy link
Contributor Author

hseok-oh commented Mar 8, 2022

cc/ @seanshpark @mhs4670go

@hseok-oh hseok-oh added the area/CI For our Continuous Integration label Mar 8, 2022
@hseok-oh hseok-oh changed the title [infra] Rremove BUILD_STAMP [infra] Remove BUILD_STAMP Mar 8, 2022
@hseok-oh hseok-oh mentioned this issue Mar 8, 2022
Merged
@hseok-oh
Copy link
Contributor Author

Removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/CI For our Continuous Integration
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hseok-oh