-
Notifications
You must be signed in to change notification settings - Fork 18
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
Importing distro files from Flatpak and Arch Linux #130
Conversation
Looks much better now, thanks! :-) |
…etch icon, gitignore for junk files
@AsciiWolf just pushed some more commits, please review. Thanks! :) |
<screenshot><image>https://raw.githubusercontent.com/hughsie/fedora-appstream/master/screenshots-extra/tremulous/d.png</image></screenshot> | ||
<screenshot><image>https://raw.githubusercontent.com/hughsie/fedora-appstream/master/screenshots-extra/tremulous/e.png</image></screenshot> | ||
<screenshot type="default"> | ||
<image type="source" width="1280" height="960">https://github.com/GrangerHub/tremulous/blob/master/misc/grangerhub-trem1.3-7d1b.jpg?raw=true</image> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a note: I may be wrong, but I think that the correct URL should be https://raw.githubusercontent.com/GrangerHub/tremulous/master/misc/grangerhub-trem1.3-7d1b.jpg
.
Same with the other screenshots/videos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this sounds right. I used to be able to get those URLs but somehow I can't seem to get to them. But just using your example works (for existing files). I will re-confirm once I merge this PR to master.
Nice, I think it looks good now (except for the screenshots URL)! The only remaining thing are the Makefile rules. :-) |
Although, we should probably sort through the screenshots, select the best ones and remove others. Currently, there are too many of them and some are very similar. For example, misc/screenshots/c-fedora.png and misc/screenshots/d-fedora.png look almost the same. |
Great! I will sort through the screenshots, but I will need some help with the Makefile. @wtfbbqhax @jkent can you help crafting an install section in the Makefile that can place binaries, icon, and desktop files in the proper places? As far as I see, we only have one target in the Makefile that just produces a zip or other file based on target distro. So we never worried about an install section. What's the situation with the CMake build? Was that ever working? Thanks! |
Looks like libcurl is also outdated and causing Linux build errors due to |
@AsciiWolf @jkent I added an @AsciiWolf I removed screenshot |
It looks good except the things that I pointed out. Thanks! |
By the way, we have a CI build failure on Linux:
Is it the same thing as already mentioned here? |
That's right, we will work on that separately, but for now you can build with:
by removing the building of the REST client. |
@AsciiWolf I think I addressed all the issues. Please check! Thanks :) |
@AsciiWolf just got around to fixing these issues. If you can check the conversations and resolve, we may be close to being done! If this is all good, I'll rebase this branch to a working release and do more testing. |
Looks very good now (except the few things mentioned before), thanks! :-) |
Any progress here? |
Co-authored-by: AsciiWolf <[email protected]>
Included more screenshots showing alien race units.
Thanks for the reminder @AsciiWolf ! I needed it. :) I added a number of alien screenshots, in addition to fixing the other issues. Thanks as usual for checking and providing feedback. Once we're done here, I think I need to update so that we can build the flatpak for 0.15. Any other next steps? |
Co-authored-by: AsciiWolf <[email protected]>
To fix curl error
@AsciiWolf removed URL from appdata as requested. I posted a comment above regarding screenshots. I also corrected the Github Actions for the Linux build, which may help with fixing the issues while building the Flatpak. |
Thanks! Everything looks good to me now. :-) |
Trying 13. macos-11 no longer supported
Co-authored-by: AsciiWolf <[email protected]>
Co-authored-by: AsciiWolf <[email protected]>
Imported packaging files and new screenshots @AsciiWolf