Bug#634738: remove wontfix tag

2022-02-15 Thread Jens Rottmann

Control: tags -1 - wontfix

Rainer Herrmann's python3.patch _did_ fix the original issue, the wontfix tag 
therefore no longer makes sense.



Bug#634738: [patch] issue no longer exists, remove reference to dead server

2022-02-15 Thread Jens Rottmann

Control: tags -1 patch

The issue described by this ticket no longer exists since ardentryst 1.71-7.

Gonéri Le Bouder reported "ardentryst.py:24: DeprecationWarning: the sha/md5 module 
is deprecated; use hashlib instead", triggered by this line:
import smtplib, email, math, sha, md5

Reiner Herrmann's Python 3 port (python3.patch) fixed the issue by replacing 
the offending line:
import smtplib, email, math
from hashlib import md5, sha1

There was also discussion how changing the code would affect uploading scores 
to the online score board, but the server has been offline since 2013 (as per 
game's author, Jordan Trudgett), so this is no longer of any interest, either.

I attached a patch removing all menu options dealing with the long-dead online 
score board. For details see the text at the start of the patch. Then I'd 
suggest closing this ticket.

Regards,
Jens

PS: I hope putting a Control command, text and patch attachment all in 1 email 
doesn't cause any issues.


offline.patch.gz
Description: application/gzip


Bug#1005849: [patch] fix 10 deprecation warnings

2022-02-15 Thread Jens Rottmann

Attached the promised patch.

deprecationfix.patch.gz
Description: application/gzip


Bug#717885: [patch] fix issues with some in-game messages

2022-02-15 Thread Jens Rottmann

Control: tags -1 patch

Attached a patch finally (after 8+ years) fixing the shop message and, while I 
was at it, another minor issue with Nyx's tutorial messages. For details see 
text at top of patch.

Regards,
Jens

messagefix.patch.gz
Description: application/gzip


Bug#1005849: ardentryst: 10 deprecation messages at various points

2022-02-15 Thread Jens Rottmann

Package: ardentryst
Version: 1.71-8
Severity: normal
Tags: patch

I encountered a total of 10 deprecation messages at various points in the game. 
These are all of the same type (implicit floar->int conversion), all 
essentially harmless, but they might look alarming to an unsuspecting player. And 
the sheer number of these warnings makes the game look unprofessional.

I'd have said Severity 'minor', but I upped it to 'normal', because there are 
so many of them. Present at least in 1.71-8 and -9. Haven't checked if they 
were introduced with Python 3 in 1.71-7 or if they were there even earlier.

I made a patch fixing all of them, they're all fairly trivial. I'll attach it 
as soon as I got the ticket number (to reference it in the patch's description).

Full list of warnings follows (line numbers are for 1.71-9):

On Resume Quest screen if >14 saves present:
/usr/share/games/ardentryst/ardentryst.py:1367: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  gsr.midleft = (70, 100 + x * 25 - scroll_factor * 25)
/usr/share/games/ardentryst/ardentryst.py:1373: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  crect.midleft = (65, 100 + cursor * 25 - scroll_factor * 25)
/usr/share/games/ardentryst/ardentryst.py:1378: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  arrowrect.center = (130, 450 + math.sin(tick/10.0)*5)
/usr/share/games/ardentryst/ardentryst.py:1384: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  handr.midright = (int(60+abs(math.sin(tick/10.0))*10.0), 100 + cursor * 25 - 
scroll_factor * 25)

At Eternal Boulder (save location):
/usr/share/games/ardentryst/ardentryst.py:2339: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  m_r.midleft = (320-Fonts[16].size(msg)[0]/2, 350)
/usr/share/games/ardentryst/ardentryst.py:2347: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  npi_r.center = (320, 400+abs(math.sin(tick/10.0)*15))
/usr/share/games/ardentryst/ardentryst.py:2383: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  m_r.midleft = (320-Fonts[16].size(msg)[0]/2, 350+y)

When Snodom reached on the map:
/usr/share/games/ardentryst/ardentryst.py:2745: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  screen.blit(Data.images["kiripan.png"][0], (290, 141 - Game.KIRI_HEIGHT*60 + 
math.sin(tick/30.0)*6))
/usr/share/games/ardentryst/ardentryst.py:2778: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  screen.blit(Data.images["kiripan.png"][0], (290, 141 - Game.KIRI_HEIGHT*60 + 
math.sin(tick/30.0)*6))

When Kiripan floats down (endscene):
/usr/share/games/ardentryst/ardentryst.py:2835: DeprecationWarning: an integer 
is required (got type float).  Implicit conversion to integers using __int__ is 
deprecated, and may be removed in a future version of Python.
  screen.blit(Data.images["kiripan.png"][0], (290, 81 + Game.ENDSCENE))

Regards,
Jens



Bug#733469: Can I add a "patch" tag like this?

2022-02-10 Thread Jens Rottmann

Control: tags -1 patch



Bug#733469: Patch to fix both issues

2022-02-10 Thread Jens Rottmann

Tags: patch

I dug deeper, turns out I was wrong, the 2 crashes when renaming or deleting 
saves weren't related after all.

Anyway, I made a patch fixing both of them. Note that it's meant to be applied 
_after_ my python3fix.patch for bug #1001823.

savesfix.patch.gz
Description: application/gzip


Bug#733469: Deleting save also exits

2022-02-10 Thread Jens Rottmann

If you have 2 or more saves, deleting one of them (with Shift-Del in the "Resume 
quest" menu) works fine. But deleting the _last_ save exits the program with two 
exceptions:

Traceback (most recent call last):
  File "/usr/share/games/ardentryst/ardentryst.py", line 4722, in 
main()
  File "/usr/share/games/ardentryst/ardentryst.py", line 4416, in main
Game = Game_SlotMenu()
  File "/usr/share/games/ardentryst/ardentryst.py", line 1582, in Game_SlotMenu
gameobj = pickle.load(open(os.path.join(SAVEDIRECTORY, "Saves", 
gamelist[cursor]),"rb"))
IndexError: list index out of range
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/share/games/ardentryst/ardentryst.py", line 4726, in 
handleException(e)
  File "/usr/share/games/ardentryst/ardentryst.py", line 685, in handleException
open("bugreport.txt", "w").write(open(os.path.join(SAVEDIRECTORY, "log.txt"), 
"r").read())
PermissionError: [Errno 13] Permission denied: 'bugreport.txt'

I'm adding this info here, because I suspect the 2 issues with 
renaming/deleting saves might have a common cause? (Note that both have been 
present long before the Python3 port, but continue to exist afterwards.)



Bug#1001823: Ping?

2022-02-08 Thread Jens Rottmann

Would you like me to provide additional info or reformat my patch?



Bug#718883: Both IndexErrors still occur in 1.71-8

2021-12-17 Thread Jens Rottmann

Tested with 1.71-8: latest package still affected by _both_ IndexError 
instances.



Bug#718883: Another way to trigger IndexError on LEVEL.map[][], same file, different code line

2021-12-17 Thread Jens Rottmann

A very similar bug, resulting in a crash, can be triggered by Nyx firing her 
crossbow. Even in the very first level, if unlucky, no additional equipment 
neccessary. Here it's line 5325, but in the same file and same mode of failure: I 
assume arrow.y becomes negative, which the code fails to check for => likewise 
IndexError trying to access LEVEL.map[][].

Fri Dec 17 19:25:21 2021| Playlevel called: Castle Centre
Traceback (most recent call last):
  File "/usr/share/games/ardentryst/ardentryst.py", line 4722, in 
main()
  File "/usr/share/games/ardentryst/ardentryst.py", line 4423, in main
handle_game(Game, True)
  File "/usr/share/games/ardentryst/ardentryst.py", line 3083, in handle_game
Result = playlevel(player, level, [levelscript, npcscript], screen, Data, 
Fonts, soundbox, Game,
  File "/usr/share/games/ardentryst/play_level.py", line 2886, in playlevel
PLAYER.arrow_tick()
  File "/usr/share/games/ardentryst/play_level.py", line 5325, in arrow_tick
lm = LEVEL.map[int(arrow.x/40)][int(arrow.y/40)]
IndexError: list index out of range



Bug#1001823: Re-send patch, bugtracker messed up spaces

2021-12-17 Thread Jens Rottmann

Turns out bugtracker messes up whitespace in text/patch attachments. :(
Re-attaching same patch, but gziped, hoping it's left intact now.


python3fix.patch.gz
Description: application/gzip


Bug#1001823: Improved patch

2021-12-17 Thread Jens Rottmann

My fix removes the only occurrence of cmp() in magic.py, so we should drop the 
line importing past.builtins.cmp. Updated patch accordingly, also improved 
commit message.

Attached patch v2, hoping bugtracker can handle attachments.
Python 3 port introduced at least five severe bugs.

* Customizing action keys via Options, Control prints 2 deprecation warnings
  and no longer works, changed keys aren't saved. Not being able to remap
  keys makes it hard for users of non-English keyboards. (controls.py)
* Playing as Nyx and casting Ice ([D][S]) or Implosion ([D][W]) instantly
  crashes the game due to 3 exceptions. (magic.py)
* Playing as Pyralis and using Spin-slash ([C][S]) hangs gameplay, you have
  to abort the current level to be able to move again. (play_level.py)

Bug-Debian: https://bugs.debian.org/1001823
Signed-off-by: Jens Rottmann

--- ardentryst/controls.py
+++ python3fix/controls.py
@@ -35,7 +35,7 @@
 class SET2:
 def __init__(self, keycodes, x, y, set):
 self.set = set
-self.keys = [Key(keycodes[c], x + ((8-c)%3) * 32, y + (c/3)*30, "B-" + str(((11-c)/3)*3 - (11-c)%3), set) for c in range(9)]
+self.keys = [Key(keycodes[c], x + ((8-c)%3) * 32, y + (c//3)*30, "B-" + str(((11-c)//3)*3 - (11-c)%3), set) for c in range(9)]

 class Key:
 def __init__(self, keycode, x, y, binding, set):
--- ardentryst/magic.py
+++ python3fix/magic.py
@@ -21,7 +21,6 @@

 import pygame, math, random
 from pygame.locals import *
-from past.builtins import cmp

 def ground_at(LEVEL, x, f=False):
 "Finds the y co-ordinate of the ground at position x."
@@ -236,7 +235,7 @@ class Ice_1(Spell):
 def s_init(self):
 global DATA
 self.affected = []
-self.cant = self.caster.mp < 4
+self.cant = self.caster.mp[0] < 4
 def s_blit(self, surf, ALT_X, ALT_Y):
 global DATA
 if not self.affected:
@@ -441,7 +440,7 @@ class Implosion_1(Spell):
 def s_init(self):
 global DATA
 self.affected = []
-self.cant = self.caster.mp < 15
+self.cant = self.caster.mp[0] < 15
 def s_blit(self, surf, ALT_X, ALT_Y):
 global DATA
 pic = DATA.mag_images["bubble.png"][0]
@@ -472,7 +471,7 @@ class Implosion_1(Spell):
 if self.caster.mp[0] >= 15:
 self.affected.append(monster)

-self.affected.sort(lambda x, y: cmp(y.maxhp, x.maxhp))
+self.affected.sort(key=lambda x: -x.maxhp)

 if len(self.affected):
 self.affected = self.affected[:1]
--- ardentryst/play_level.py
+++ python3fix/play_level.py
@@ -4727,7 +4727,7 @@ class Character:
 self.mycombotime -= 1
 if self.chainmove[1] and self.chainmove[1] > 0:
 self.chainmove[1] -= 1
-if self.chainmove[1] and self.chainmove[1] == 0:
+if self.chainmove[1] == 0:
 cm = self.chainmove[:]
 self.chainmove = [None, None]
 getattr(self, cm[0])()
--


Bug#1001823: Ardentryst near-unplayable after Python3 port

2021-12-16 Thread Jens Rottmann

Package: ardentryst
Version: 1.71-8
Severity: grave
Tags: patch

After Ardentryst was ported to Python 3 it appearently hasn't been play-tested 
well, several common actions don't work or crash the game, making it 
near-unplayable.

Start a new game, playing as "Nyx". Enter the 1st level. Try to cast "Ice" by pressing 
[D], [S]. Game crashes, printing a backtrace. Ice is Nyx's starting spell, the tutorial even instructs the 
player to try it, so it's quite hard to get through the game without it. Especially as her higher level spell 
"Implosion" insta-crashes, too.

For 2 more bugs, see below. Patch follows:

--X8-X8
Python 3 port caused several severe bugs.

* Customizing action keys via Control Options no longer works, changed
  keys aren't saved. Also a deprecation warning is printed. (controls.py)
* Playing as Nyx and casting Ice ([D][S]) or Implosion ([D][W]) instantly
  crashes the game with an exception. (magic.py)
* Playing as Pyralis and using Spin-slash ([C][S]) hangs gameplay, you have
  to abort the current level to be able to move again. (play_level.py)

Signed-off-by: Jens Rottmann

--- ardentryst/controls.py
+++ py2-3fixes/controls.py
@@ -35,7 +35,7 @@
 class SET2:
 def __init__(self, keycodes, x, y, set):
 self.set = set
-self.keys = [Key(keycodes[c], x + ((8-c)%3) * 32, y + (c/3)*30, "B-" + 
str(((11-c)/3)*3 - (11-c)%3), set) for c in range(9)]
+self.keys = [Key(keycodes[c], x + ((8-c)%3) * 32, y + (c//3)*30, "B-" 
+ str(((11-c)//3)*3 - (11-c)%3), set) for c in range(9)]

 class Key:
 def __init__(self, keycode, x, y, binding, set):
--- ardentryst/magic.py
+++ py2-3fixes/magic.py
@@ -236,7 +236,7 @@
 def s_init(self):
 global DATA
 self.affected = []
-self.cant = self.caster.mp < 4
+self.cant = self.caster.mp[0] < 4
 def s_blit(self, surf, ALT_X, ALT_Y):
 global DATA
 if not self.affected:
@@ -441,7 +441,7 @@
 def s_init(self):
 global DATA
 self.affected = []
-self.cant = self.caster.mp < 15
+self.cant = self.caster.mp[0] < 15
 def s_blit(self, surf, ALT_X, ALT_Y):
 global DATA
 pic = DATA.mag_images["bubble.png"][0]
@@ -472,7 +472,7 @@
 if self.caster.mp[0] >= 15:
 self.affected.append(monster)

-self.affected.sort(lambda x, y: cmp(y.maxhp, x.maxhp))
+self.affected.sort(key=lambda x: -x.maxhp)

 if len(self.affected):
 self.affected = self.affected[:1]
--- ardentryst/play_level.py
+++ py2-3fixes/play_level.py
@@ -4727,7 +4727,7 @@
 self.mycombotime -= 1
 if self.chainmove[1] and self.chainmove[1] > 0:
 self.chainmove[1] -= 1
-if self.chainmove[1] and self.chainmove[1] == 0:
+if self.chainmove[1] == 0:
 cm = self.chainmove[:]
 self.chainmove = [None, None]
 getattr(self, cm[0])()
--



Bug#821426: 'Show Global Variables' doesn't work

2016-04-18 Thread Jens Rottmann
Package: nemiver
Version: 0.9.5-2

Debian 8.4 amd64, gdb 7.7.1+dfsg-5.

Compiled this with gcc -O0 -g:
int x = 5;
int main() { return x; }

In nemiver selecting "Show Global Variables" prints this error message:

|E|virtual void
nemiver::OnGlobalVariablesListedHandler::do_handle(nemiver::CommandAndOutput&):/tmp/buildd/nemiver-0.9.5/src/dbgengine/nmv-gdb-engine.cc:2299:failed
to extract global variable list

the "Global Variables" window remains empty, i.e. list header and close
button, but no variable data shown.

Maybe the GDB/MI API changed between versions i.e. between Debian's gdb
and whatever gdb the nemiver author coded against? Just guessing...

Thanks,
Jens



Bug#720599: libkmod2: modprobe -f doesn't work, fixed in kmod 12

2013-08-23 Thread Jens Rottmann
Package: libkmod2
Version: 9-3
Severity: normal
Tags: fixed-upstream

In kmod before kmod 12 modprobe -f (force loading of module from a different 
version kernel) has no useful effect. It just changes the error message that 
the version info doesn't match to another error message that the version info 
is missing.

As of writing, Debian Wheezy, Jessie and Sid all ship kmod 9, which is 
affected. A workaround is to downgrade to module-init-tools from Squeeze.

The bug was fixed in kmod 12 upstream, please upgrade:
http://article.gmane.org/gmane.linux.hotplug.devel/17644

Many thanks,
Jens


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#495795: dropbear: please provide the scp binary

2012-07-24 Thread Jens Rottmann
Seconded.

Now Wheezy is frozen.  :-(

Could scp be added to Sid at least, please?

Thanks,
Jens


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org