aria2: appletls is not available on 10.6 or earlier

[kencu]

No description provided.

[larryv]

You should just use

if {${os.platform} eq "darwin" && ${os.major} <= 10} {

and avoid the excess nesting.

[kencu]

I like that! Thanks.

[kencu]

Please commit it as it is and I'll consider larry's changes for the next portfile fix I suggest.

[neverpanic]

While I don't object merging this as-is, isn't the point of having pull request reviews that the comments can be fixed before the code is merged?

[larryv]

Uh, yes. Yes it is. @ken-cunningham-webuse, why don’t you want to make the change?

[Schamschula]

I agree with @neverpanic. Please add @larryv's change.

[kencu]

OK. I found a reference how to do this:

http://stackoverflow.com/questions/16748115/how-to-modify-github-pull-request

[larryv]

Aesthetics are up to the maintainer, who is @Schamschula.

You just have to modify your branch; GitHub will pick up the new commits automatically.

[kencu]

I think that actually worked...

[Schamschula]

Looks good.

[raimue]

This is an example of a pull request that should have been squashed before merging. As a separate commit, without any connection to this pull request, the commit message does not make much sense.