all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Tadeus Prastowo <tadeus.prastowo@unitn.it>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: 35056@debbugs.gnu.org
Subject: bug#35056: 26.1; Tetris score is no longer updated due to commit 995be66f0f0d26d1a96cbb8dfb429c3941157771
Date: Sun, 31 Mar 2019 21:41:20 +0200	[thread overview]
Message-ID: <CAN-HRFZH36rHKEU9KLvbGTcBFYmvB+1rY5NMOzVwC-GfZ2UOcQ@mail.gmail.com> (raw)
In-Reply-To: <78ea6992-85b1-e005-2c33-7b1df1c04031@cs.ucla.edu>

On Sun, Mar 31, 2019 at 5:32 AM Paul Eggert <eggert@cs.ucla.edu> wrote:
>
> Apparently the problem is that there are two formats of Tetris game-score files.
> One format is used by play/gamegrid.el itself, the other is used by the
> update-game-score command. I didn't know that; I thought that the only reason to
> use update-game-score was to get setuid/setgid permissions, and that the file
> format didn't change.

Perhaps you need to make a test unit to ensure there is no regression
in the future.

> To some extent the file-format problem is Just One Of Those Things: if up
> upgrade Emacs and it changes the file format, Emacs is not smart enough to deal
> with the format change and so it loses your old Tetris scores.
>
> That being said, Emacs could try harder to use the update-game-score program if
> it's installed. Is it in your installation? If so, please try the attached
> patch. If not, please try redoing 'configure' so that update-game-score is built
> and installed, and then try the attached patch.

I apply your patch to Emacs 26.1 source
(http://ftp.gnu.org/gnu/emacs/emacs-26.1.tar.xz), and run the
following configure command:
configure --prefix=/mnt/core/Codebases/installzone/emacs
--with-xwidgets --disable-silent-rules --with-gameuser=yes
And, then the following make command:
make
make install

I played Tetris, scoring higher than the lowest recorded in my high
score file, and I confirm that your patch allows the score to be
recorded after year 2018 went by without any new record due to the bug
introduced by commit 995be66f0f0d26d1a96cbb8dfb429c3941157771.

> I suppose it would be possible to do more-serious hacking in gamegrid.el to deal
> with the incompatible file formats (convert formats back and forth, say), but
> that would be beyond my call of duty.

I like your patch: if it ain't broken, don't fix it, dude ;)

--
Best regards,
Tadeus





  reply	other threads:[~2019-03-31 19:41 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-30 22:32 26.1; Tetris score is no longer updated due to commit 995be66f0f0d26d1a96cbb8dfb429c3941157771 Tadeus Prastowo
2019-03-31  3:32 ` bug#35056: " Paul Eggert
2019-03-31 19:41   ` Tadeus Prastowo [this message]
2019-03-31 22:54   ` Richard Stallman
2019-04-01  3:03     ` Paul Eggert

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAN-HRFZH36rHKEU9KLvbGTcBFYmvB+1rY5NMOzVwC-GfZ2UOcQ@mail.gmail.com \
    --to=tadeus.prastowo@unitn.it \
    --cc=35056@debbugs.gnu.org \
    --cc=eggert@cs.ucla.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.