Skip to content
This repository has been archived by the owner on Sep 23, 2024. It is now read-only.

Expand basecfg path earlier #129

Merged
merged 1 commit into from
May 15, 2018

Conversation

major
Copy link
Contributor

@major major commented May 14, 2018

Expanding the basecfg path earlier ensures that any function that uses the path to the base kernel config is getting the same absolute path to the file.

Partial fix for #94.

@major major added the enhancement New feature or request label May 14, 2018
@major major self-assigned this May 14, 2018
@major major requested a review from veruu May 14, 2018 16:24
@coveralls
Copy link

coveralls commented May 14, 2018

Pull Request Test Coverage Report for Build 435

  • 1 of 3 (33.33%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.06%) to 47.006%

Changes Missing Coverage Covered Lines Changed/Added Lines %
skt/executable.py 0 2 0.0%
Totals Coverage Status
Change from base Build 432: -0.06%
Covered Lines: 683
Relevant Lines: 1453

💛 - Coveralls

@major major force-pushed the expand-basecfg-paths-early branch 4 times, most recently from 0e7bffd to 31d36f8 Compare May 14, 2018 17:04
Move the path expansion for the kernel configuration file to the
load_config() function.

Signed-off-by: Major Hayden <[email protected]>
@major major force-pushed the expand-basecfg-paths-early branch from 31d36f8 to 791cad0 Compare May 14, 2018 17:38
@danghai
Copy link
Contributor

danghai commented May 14, 2018

@major , do you also need to have full path rc too?

@major
Copy link
Contributor Author

major commented May 14, 2018

@danghai Yes, but I will put that into a different PR.

@danghai
Copy link
Contributor

danghai commented May 14, 2018

@major Could I take this with this commit: danghai@5cd7c3f ?

@veruu veruu merged commit 77b3f69 into cki-project:master May 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants