Bug#634738: Fwd: Debian bug#634738: Ardentryst, deprecation, checksums and obfuscate code

[Vincent Cheng]

Forgot to forward upstream's reply to the BTS on his behalf, so doing so now.

Vincent


---------- Forwarded message ----------
From: Jordan <jordan_trudgett at hotmail.com>
Date: Mon, Jul 8, 2013 at 7:38 PM
Subject: Re: Debian bug#634738: Ardentryst, deprecation, checksums and
obfuscate code
To: Vincent Cheng <vincentc1208 at gmail.com>


Ardentryst is no longer under development and high score boards will
be closed in the near future (this year.)

Sorry if this email is short, sent from my phone :-)


----- Reply message -----
From: "Vincent Cheng" <vincentc1208 at gmail.com>
To: "Steve Cotton" <steve at s.cotton.clara.co.uk>
Cc: "Jordan Trudgett" <jordan_trudgett at hotmail.com>
Subject: Debian bug#634738: Ardentryst, deprecation, checksums and
obfuscate code
Date: Tue, Jul 9, 2013 10:16 am


Hi Steve,

[cc-ed Jordan (Ardentryst's upstream dev)]

On Mon, Jul 8, 2013 at 6:37 AM, Steve Cotton <steve at s.cotton.clara.co.uk> wrote:
> Hi Vincent,
>
> I'm looking at deprecation warnings that are generated when
> Ardentryst is run on modern Python.  This is Debian bug 634738,
> which I see you looked at a while ago.
>
> As a reminder, there's a tiny section of lightly obfuscated code
> in Ardentryst to make it harder to cheat on the leaderboard.  The
> deprecation warnings point to that code.  Also, any changes to the
> code invalidate scores from being submitted to the leaderboard.
>
> From a technical side, I can provide a patch.  But I'm not clear
> on whether it would be better to just make the bug a WONTFIX.
>
> I've deliberately not CC'd the bug report, as I'd like to get your
> and upstream's opinion on the obfuscated checksumming first.  But
> please feel free to include this message in a mail to the BTS.
>
> Thanks in advance,
> Steve

AFAIK upstream already knows about this issue (Jordan, for context
this is about runtime DeprecationWarnings due to the use of python's
sha and md5 modules in ardentryst; see http://bugs.debian.org/634738).
I don't think upstream is actively working on the original ardentryst
anymore (please correct me if I'm wrong), but if you have any patches
on hand, please send them directly upstream.

I'm personally not all that concerned about this bug, and I'd rather
just tag this as wontfix in Debian's BTS unless you or upstream have
any objections.

Cheers,
Vincent