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

unifi:Update download location #11621

Merged
merged 1 commit into from
Dec 13, 2015
Merged

unifi:Update download location #11621

merged 1 commit into from
Dec 13, 2015

Conversation

Baughn
Copy link
Contributor

@Baughn Baughn commented Dec 11, 2015

The old link's broken.

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @wkennington to be a potential reviewer

@wkennington
Copy link
Contributor

Any reason? Does the old link no longer work?
On Thu, Dec 10, 2015 at 7:26 PM Svein Ove Aas [email protected]
wrote:


You can view, comment on, or merge this pull request online at:

#11621
Commit Summary

  • Update download location

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#11621.

@bjornfor
Copy link
Contributor

Pleas update commit message: ": "

@Baughn
Copy link
Contributor Author

Baughn commented Dec 11, 2015

Yes, the old link no longer works.

...Github doesn't seem to give me a way of changing the commit message.

@Baughn Baughn changed the title Update download location unifi:Update download location Dec 11, 2015
@jagajaga
Copy link
Member

You need to do

git rebase -i HEAD~1
# do reword there
git push --force <your remote> <your branch>

@Baughn
Copy link
Contributor Author

Baughn commented Dec 11, 2015

Well, I made the patch by using the edit button on github...

..I'll fix up the commit title once I've got a working dev environment again.

@Baughn Baughn force-pushed the patch-2 branch 2 times, most recently from c7f6c21 to 4a33cce Compare December 11, 2015 22:24
@Baughn
Copy link
Contributor Author

Baughn commented Dec 11, 2015

Fix'd.

Do I need to do anything special for this to also make it into the 15.09 channel?

cillianderoiste added a commit that referenced this pull request Dec 13, 2015
unifi:Update download location
@cillianderoiste cillianderoiste merged commit 4ce64a9 into NixOS:master Dec 13, 2015
@cillianderoiste
Copy link
Member

Thanks @Baughn, I'll cherry-pick it to 15.09.

@cillianderoiste
Copy link
Member

I had to upgrade the version on 15.09 too because the old version is no longer available.

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.

6 participants