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

Commit mini-db #9728

Merged
merged 5 commits into from Feb 6, 2021
Merged

Commit mini-db #9728

merged 5 commits into from Feb 6, 2021

Conversation

Nekotekina
Copy link
Member

@Nekotekina Nekotekina commented Feb 5, 2021

I want the ability to see commits associated with releases in RPCS3 interface. Recent commits can be fetched via github API, but I believe it still needs a base to not fetch more than necessary. Base can be updated lazily (by appending new commits).

Why manipulating resources, added some additions for Linux builds (AppImage, custom out of source builds) so they seem to be able to find themes.

@Nekotekina Nekotekina changed the title Add a commit mini-db (unused) Commit mini-db Feb 5, 2021
Copy link
Contributor

@Megamouse Megamouse left a comment

Choose a reason for hiding this comment

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

Hmm. I wonder why we need that when you can just look it up in any git frontend.

My main issue codewise is that you bloat main for no reason.
Just put it in another file.
Also, why with cli ?

rpcs3/main.cpp Outdated Show resolved Hide resolved
@Nekotekina
Copy link
Member Author

Nekotekina commented Feb 5, 2021

I used CLI to create initial commit db because anonymous limit is 60 per hour. But once it's written it will only need to be rarely appended. It's also seems convenient to update from CLI if put in some script I use.

Located in bin/git/commits.lst;
Created with GitHub API with --get-commit-db key.
Can be lazily appended with the same key.
@Nekotekina
Copy link
Member Author

Nekotekina commented Feb 5, 2021

I think that code can be moved out of main.cpp when it's properly (re)factored for actual use. The problem is, the information is spread, git frontend isn't installed, etc etc. Accessing old PR data is non-trivial, it's also hard to gather all information.

@YuriNator557
Copy link

RPCS3.log
RPCS3 is now crashing on startup.

@GarettJaxx
Copy link

GarettJaxx commented Feb 6, 2021

RPCS3.log

Agreed - i now get instacrash upon opening RPCS3 after updating - error shown is:
image

@Nekotekina
Copy link
Member Author

Oh sorry, I think I'll fix it right now.

@@ -209,6 +209,12 @@ bool update_manager::handle_json(bool automatic, bool check_only, const QByteArr
m_expected_hash = latest[os]["checksum"].toString().toStdString();
m_expected_size = latest[os]["size"].toInt();

if (!m_request_url.starts_with("https://github.com/RPCS3/rpcs3"))
Copy link
Member

@AniLeo AniLeo Feb 6, 2021

Choose a reason for hiding this comment

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

Dangerous change, will break the update on every old build if we're ever forced to do another build host migration such as the previous GitHub -> AppVeyor

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.

None yet

5 participants