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

Added icons for RC3 and RC4 #170

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Added icons for RC3 and RC4 #170

merged 1 commit into from
Feb 5, 2024

Conversation

saltedlolly
Copy link

Quickly did these by editing the existing logo file. We could replace rc3 logo and replace the release. At least then people know what version they are running.

@ghost
Copy link

ghost commented Feb 4, 2024

Good work @saltedlolly !
Screenshot from 2024-02-04 19-43-04

Copy link
Member

@ycagel ycagel left a comment

Choose a reason for hiding this comment

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

ACK.

@ycagel ycagel requested a review from a user February 5, 2024 19:50
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

ACK.

Copy link

@JaredTate JaredTate left a comment

Choose a reason for hiding this comment

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

ACK. Compiles, runs & looks great! Thanks for doing this and thinking ahead with RC4. I canceled my Adobe membership last fall to save some $$, so I don't have an easy way to edit these.

Screenshot 2024-02-05 at 3 09 30 PM

@JaredTate
Copy link

ACK.

Looks like you need to hit the "review changes" in the upper right-hand corner to "approve" so it provides a 3rd check mark approval. Thanks for helping with reviews & all your work!

@ghost
Copy link

ghost commented Feb 5, 2024

Screenshot from 2024-02-05 23-13-50

i did but my check is not the same color ^^

Screenshot from 2024-02-05 23-16-28

Update: now its blue! ty

@JaredTate
Copy link

JaredTate commented Feb 5, 2024

Screenshot from 2024-02-05 23-13-50

i did but my check is not the same color ^^

Screenshot from 2024-02-05 23-16-28

Update: now its blue! ty

It should work now, there was a permissions issue in the organization. Cheers

@saltedlolly
Copy link
Author

@Jongjan88 and myself are only reviewers so our approvals never count towards one of the three required approvals. I am okay with that as I am not a C++ dev (though I would not approve anything I don't understand). On the other hand Jan is a C++ dev so he IS qualified to review changes, if and when the core devs want to allow him that responsibility.

@JaredTate
Copy link

@Jongjan88 and myself are only reviewers so our approvals never count towards one of the three required approvals. I am okay with that as I am not a C++ dev (though I would not approve anything I don't understand). On the other hand Jan is a C++ dev so he IS qualified to review changes, if and when the core devs want to allow him that responsibility.

He should be good to go now, and I have no problem adding you as a full reviewer either. You have contributed a ton and are very active. I will ask the other guys what they think. It would be great to have more reviewers for continuous development.

@JaredTate JaredTate merged commit 3c77dec into DigiByte-Core:develop Feb 5, 2024
@saltedlolly
Copy link
Author

@JaredTate I was trying to figure out how to help fix the build process to ensure the filenames and folder names are correct, as previously discussed. Is it in this repo, or somewhere else? https://github.com/DigiByte-Core/DigiByte-GUIX-Docker

@JaredTate
Copy link

That is for the docker container, not the actual GUIX build process. I think the code for filenames is here, but not 100% sure where the folder names are drawn from in GUIX code. The file names change again after the complete formal signing process. Part of the signing process for a formal release includes the commit hash in the filename/folder to verify the commit the binaries are compiled from.

# Usage: outdir_for_host HOST SUFFIX
#
# HOST: The current platform triple we're building for
#
outdir_for_host() {
echo "${OUTDIR_BASE}/${1}${2:+-${2}}"
}
# Usage: profiledir_for_host HOST SUFFIX
#
# HOST: The current platform triple we're building for
#
profiledir_for_host() {
echo "${PROFILES_BASE}/${1}${2:+-${2}}"
}

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.

3 participants