Display SHA1 sum for downloadable packages. about openraweb HOT 7 CLOSED

This looks like as if the patch was generated with git. Why not simply send a pull request, @erlehmann?

from openraweb.

Matthias Mailänder [email protected] writes:

This looks like as if the patch was generated with git. Why not simply
send a pull request, @erlehmann?

Because the output of “git request-pull” is a bit longer, but not much
more helpful than a simple patch file.

Nils Dagsson Moskopp // erlehmann
http://dieweltistgarnichtso.net

from openraweb.

Welcome to the 21st century where we don't exchange patches via mailing lists anymore. =) Seriously, you will also get credited properly in the commit history if you file a pull request yourself so I would advise that. See https://github.com/OpenRA/OpenRAWeb/pulls

from openraweb.

On Tue, Jan 27, 2015 at 7:57 AM, Matthias Mailänder <
[email protected]> wrote:

Welcome to the 21st century where we don't exchange patches via mailing
lists anymore. =) Seriously, you will also get credited properly in the
commit history if you file a pull request yourself so I would advise that.

Well... about that (pretty much all the critical bits of your system are
still developed that way).

—
Reply to this email directly or view it on GitHub
#182 (comment).

from openraweb.

I guess git then has a patch import functionality to do that more conveniently?

from openraweb.

It does -- you can apply an mbox trivially.

We're getting way off topic now though :)

Let's:

• Have someone (Taryn?) make a PR for this so our normal workflow can be
  used.
• Consider the technical merits of the patch.

On the second point, I'm not a huge believer in delivering SHA1sums
alongside packages UNLESS it's all delivered over HTTPS. It doesn't provide
any meaningful assurance of non-tampering otherwise.

I don't think this is actively harmful though, so a tentative +1.

On Tue, Jan 27, 2015 at 9:21 AM, Matthias Mailänder <
[email protected]> wrote:

I guess git then has a patch import functionality to do that more
conveniently?

—
Reply to this email directly or view it on GitHub
#182 (comment).

from openraweb.

The problem here is finding a place to put it without messing up the size of the buttons.
It can live on the tooltip, but thats not useful enough to justify its existence (terrible discovery and can't copy the sha1). I'm tempted to say WONTFIX unless somebody has a concrete suggestion on how to make this work.

from openraweb.