search

ymattw / ydiff

View colored, incremental diff in workspace or from stdin, side by side and auto paged.
Other
877 stars 62 forks source link

Fedora package #78

Closed AloisMahdal closed 5 years ago

AloisMahdal commented 6 years ago

With the comand conflict resiolved, I took the first step to get slop into Fedora. It's my second package ever, so :crossed_fingers:!

Once (I mean, if) it succeeds, this issue can serve to add the info to the README.rst.

AloisMahdal commented 6 years ago

So package review has been completed, dist repo created. Unless someone finds bug, it's basically just matter of time when ydiff will be available in Fedora:

AloisMahdal commented 6 years ago

Also, if someone wants to help with testing on F27/28, you can already install ydiff now using following command:

dnf install ydiff --enablerepo=updates-testing

If you find any Fedora-specific issues, add comment to the respective Bodhi instance above (or just comment here, if you don't have Fedora account).

ymattw commented 6 years ago

Thanks for your effort! I plan to release perforce support in near future, what will be the process to get the package updated for Fedora too?

AloisMahdal commented 6 years ago

TL;DR: Just release ydiff as you normally would; I'll take care of the rest.

Long answer: It's part of my role as packager to watch for upstream releases and update Fedora package in a timely manner (which I'll be happy to do). Unless there are unusual new dependencies, packaging changes or regression, I should be done with the new package in few minutes. Then it goes to update testing phase (Bodhi), where it stays for couple of weeks (2, IIRC) to allow for testing; if no issues are found it gets auto-approved and synced to stable repos (normally all F27/F28/... at once). (Rawhide skips the Bodhi process, so it's included overnight after I submit the update.)

AloisMahdal commented 6 years ago

...aaand, the package is now in stable.

ymattw commented 6 years ago

Awesome, thank you!

On Thu, Jul 12, 2018 at 1:25 AM Alois Mahdal notifications@github.com wrote:

...aaand, the package is now in stable.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ymattw/ydiff/issues/78#issuecomment-404341128, or mute the thread https://github.com/notifications/unsubscribe-auth/ABV4jy0mjG-obqiLrLqjTtSnv-XBVZuZks5uFolngaJpZM4U4vFC .