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 possibility to use alternate download locations (sparkle:os attribute) #14

Merged
merged 2 commits into from
Jan 27, 2014

Conversation

alexnovichkov
Copy link

Added possibility to use alternate download locations in different operating systems
by specifying sparkle:os attribute in appcast.xml file.

Appcast.xml can contain one for Mac OS X
and one with <enclosure sparkle:os="windows" ...> for Windows.

If no sparkle:os="windows" attribute found, the first is used as usual.

@vslavik
Copy link
Owner

vslavik commented Jan 23, 2014

Wow, that was fast, thank you!

However, do you think you could reformat your changes to use 4 spaces instead of tabs, as the rest of the sources (see the diff for what mixing them did to readability)? No other comments except these nitpicks, looks great to me otherwise.

TIA!

@alexnovichkov
Copy link
Author

Yeah, sure, but maybe tomorrow?

2014/1/23 Václav Slavík notifications@github.com

Wow, that was fast, thank you!

However, do you think you could reformat your changes to use 4 spaces
instead of tabs, as the rest of the sources (see the diff for what mixing
them did to readability)? No other comments except these nitpicks, looks
great to me otherwise.

TIA!


Reply to this email directly or view it on GitHubhttps://github.com//pull/14#issuecomment-33140807
.

fixed tabs, added parsing break in OnEndElement()
vslavik added a commit that referenced this pull request Jan 27, 2014
Add sparkle:os attribute support for multi-platform feeds.
@vslavik vslavik merged commit 360c589 into vslavik:master Jan 27, 2014
@vslavik
Copy link
Owner

vslavik commented Jan 27, 2014

Thanks!

@alexnovichkov alexnovichkov deleted the sparkle_os branch January 28, 2014 05:02
vslavik added a commit that referenced this pull request Jun 6, 2014
This bug was introduced by PR #14 in ec85b3c.

Fixes #24.
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.

None yet

2 participants