Bug 81086 - wrong code in windows.jl
wrong code in windows.jl
Product: Red Hat Public Beta
Classification: Retired
Component: sawfish (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Havoc Pennington
David Lawrence
: Triaged
Depends On:
Blocks: 79579
  Show dependency treegraph
Reported: 2003-01-03 20:29 EST by Gérard Milmeister
Modified: 2007-04-18 12:49 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2003-01-28 18:27:33 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Patch against sawfish-1.2 to fix windows.jl (493 bytes, text/plain)
2003-01-13 00:11 EST, Nathan G. Grennan
no flags Details

  None (edit)
Description Gérard Milmeister 2003-01-03 20:29:45 EST
From Bugzilla Helper:
User-Agent: Mozilla/5.0 Galeon/1.2.7 (X11; Linux i686; U;) Gecko/20021128

Description of problem:
The code for adjust-position-for-gravity/y
in windows.jl must be:

    (let* ((tl-off (cdr (window-frame-offset w)))

instead of:

    (let* ((tl-off (window-frame-offset w))

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:

Additional info:
Comment 1 Nathan G. Grennan 2003-01-08 10:16:42 EST
Ran into this with mplayer-0.90rc2.
Comment 2 Havoc Pennington 2003-01-12 22:59:51 EST
Upstream as http://bugzilla.gnome.org/show_bug.cgi?id=103319
Comment 3 Nathan G. Grennan 2003-01-12 23:50:31 EST
It isn't a feature request as your reported in the gnome bug. Without the fix
when using mplayer-0.90rc2 and pressing f for fullscreen it doesn't fullscreen
right, gives an error in .xsession-errors, and beeps(sign of error).

If you so don't want to maintain sawfish in RedHat why don't you just drop the
package and then everyone will move on.
Comment 4 Nathan G. Grennan 2003-01-13 00:00:52 EST
Here is a message to me from John Harper on the sawfish mailing list back in
November saying he fixed this in CVS. Should you could try using CVS instead of
the offical 1.2 release, like you have done in the past.

Comment 5 Havoc Pennington 2003-01-13 00:06:36 EST
We are packaging Sawfish (maintaining the spec file, tracking new versions,
passing bugs along to upstream), but not much beyond that. Some people like to
have it in RPM format on the CD. I don't see the harm in that.

I'm not sure what other changes would come with a CVS snap, so I'd prefer to
stick to something that's known to mostly work, as I don't have any firsthand
Comment 6 Nathan G. Grennan 2003-01-13 00:11:30 EST
Created attachment 89330 [details]
Patch against sawfish-1.2 to fix windows.jl
Comment 7 Nathan G. Grennan 2003-01-13 00:15:28 EST
I am one of the people that does like it included on the cd. But I would rather
have it not on the CD so that someone else actually does maintain it at
something like FreshRPMS or FalseHope instead of letting it rott in the

I have attched the above patch in hopes that you will include it, and if not for
others who are attached to this bug report or might find it in the future.
Comment 8 Havoc Pennington 2003-01-13 00:20:19 EST
OK, will put in the patch.
Comment 9 Havoc Pennington 2003-01-28 18:27:33 EST
In 1.2-4

Note You need to log in before you can comment on or make changes to this bug.