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

[dsm6 branch] tt-rss upgrade to 17.4 #2750

Closed
wants to merge 14 commits into from
Closed

[dsm6 branch] tt-rss upgrade to 17.4 #2750

wants to merge 14 commits into from

Conversation

michailf
Copy link
Contributor

@michailf michailf commented Apr 26, 2017

Motivation: Upgrade tt-rss to version 17.4.
Includes pull requests: #2737 (updated) #2727 #2056 (updated)
Solves issues: #2278

Checklist

  • Build rule all-supported completed successfully
  • Package upgrade completed successfully
  • New installation of package completed successfully

@michailf
Copy link
Contributor Author

Technically only michailf/spksrc@7a37b1b is needed to upgrade tt-rss to 17.4

Other commits can be handled by their own pull requests.

@ymartin59
Copy link
Contributor

You said it... "other commits should be handled by their own pull requests". So please do it, create specific pull requests with clean commits - for so learn to use git amend and rebase to fix previous commits instead of adding new ones. And please, avoid to include other PR in your proposal... for testing, create local branches in your repository where you can merge dependencies your are interested in.

Copy link
Contributor

@ymartin59 ymartin59 left a comment

Choose a reason for hiding this comment

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

Really strange... I have automatically trigger "make digests" enforcing re-downloading toolchain archives. I guess they have been uploaded updated. I have updated my PR with digests for all toolchains.

@michailf
Copy link
Contributor Author

michailf commented May 2, 2017

I have created a new pull request #2753 based on master since the upgrade is not strictly related to DSM6.

@michailf michailf closed this May 2, 2017
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.

2 participants