Skip to content

Use single cache rather then reconfiguring/moving it #13209

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

Merged
merged 1 commit into from
Jan 8, 2021
Merged

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Jan 8, 2021

This was split out from the change to use a single sysroot (#13090). I
think it cleaner this way: There is single cache, with single lock file
and the root doesn't change, but the libraries live in sub-directories
within the cache.

@sbc100 sbc100 requested a review from kripken January 8, 2021 12:53
tools/cache.py Outdated

def erase_file(self, shortname):
name = os.path.join(self.dirname, shortname)
if os.path.exists(name):
logging.info('Cache: deleting cached file: %s', name)
tempfiles.try_delete(name)

def getlib(self, libname, *args, **kwargs):
Copy link
Member

Choose a reason for hiding this comment

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

get_lib with an _?

@sbc100 sbc100 force-pushed the use_single_cache branch 2 times, most recently from eb0043a to 1b4fa88 Compare January 8, 2021 19:40
This was split out from the change to use a single sysroot (#13090).  I
think it cleaner this way: There is single cache, with single lock file
and the root doesn't change, but the libraries live in sub-directories
within the cache.
@sbc100 sbc100 merged commit d0e647b into master Jan 8, 2021
@sbc100 sbc100 deleted the use_single_cache branch January 8, 2021 23:25
kripken pushed a commit that referenced this pull request Jan 25, 2021
#13209 added get_lib but it was missed in #13139.
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.

2 participants