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

Fixes an outrageous number of bugs and warnings #90

Merged
merged 23 commits into from
Jan 13, 2025
+524 −325
Merged

Fixes an outrageous number of bugs and warnings #90

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
d964188
Fixes an outragous number of bugs and warnings
viniciusferrao Jan 8, 2025
233c439
Fixed more outrageous bugs.
viniciusferrao Jan 9, 2025
d7dcb49
Disable cosmetic warnings
Jan 9, 2025
61a01f3
Pin spdlog version
Jan 9, 2025
527bd42
Fix warning
Jan 9, 2025
2de5a13
Run clang-format
Jan 9, 2025
b2c1bb6
Handle copy to temporary file error
Jan 9, 2025
75c03af
Fix message in CMakeLists.txt
Jan 9, 2025
a892e84
Handle error in answerfile minor version
Jan 9, 2025
cc06a6c
Fix routing error disabling the VM network
Jan 9, 2025
1f55fdf
Enable Managed network again
Jan 9, 2025
8712198
Fix answerfile version handling
Jan 9, 2025
fb6231a
Fix copyFile implementation
Jan 10, 2025
3dfb889
Fix typos, change variables names and fix throw.
viniciusferrao Jan 10, 2025
0bfe8ee
Clearning more warnings
Jan 10, 2025
121594c
Fix the last warnings
Jan 11, 2025
5c3012a
Remove std::exit call, and raise an exception instead
Jan 11, 2025
66ce55b
Run clang-format
Jan 13, 2025
a659471
Added version ranges to conanfile.txt
viniciusferrao Jan 13, 2025
69d6dac
Tries to update the clang-format.yml workflow
viniciusferrao Jan 13, 2025
39ba2c1
Try to fix clang-format again
viniciusferrao Jan 13, 2025
bb0eae7
Last try to fix clang-format
viniciusferrao Jan 13, 2025
b6a6fa0
Final try to fix clang-format. I promise.
viniciusferrao Jan 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion src/repos.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,13 @@ void RepoManager::commitStatus()

for (auto const& dir_entry :
std::filesystem::directory_iterator { tmpdir }) {
std::filesystem::copy(dir_entry, "/etc/yum.repos.d");
try {
std::filesystem::copy(dir_entry, "/etc/yum.repos.d");
} catch (std::filesystem::filesystem_error const& ex) {
if (ex.code().message().starts_with("File exists")) {
Copy link
Owner Author

Choose a reason for hiding this comment

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

We must not compare string to handle errors. That's very bad design. Text can change, and the language of the glibc may also changes this. We need to aim for portability so a better implementation is to compare the error return code, like this:

if (ex.code().default_error_condition() == std::errc::file_exists) {
	continue;
}

// If it's not "file exists", maybe rethrow or handle differently
std::cerr << "Copy failed: " << ex.what() << '\n';
// handle or rethrow

continue;
}
}
}

std::vector<std::string> to_enable;
Expand Down
Loading