Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 96702 - x11-libs/vte reguest for patch inclusion
Summary: x11-libs/vte reguest for patch inclusion
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords: Inclusion
Depends on:
Blocks: 95548
  Show dependency tree
 
Reported: 2005-06-21 07:03 UTC by Zarick Lau
Modified: 2005-06-27 09:42 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
modified ebuild for 0.11.12 (vte-0.11.12.ebuild,1020 bytes, text/plain)
2005-06-21 07:04 UTC, Zarick Lau
Details
Modified ebuild for 0.11.13 (vte-0.11.13.ebuild,1.05 KB, text/plain)
2005-06-21 07:04 UTC, Zarick Lau
Details
Patch 1 (save space for each new terminal) (vte-0.11.12-spaces1.patch,17.13 KB, patch)
2005-06-21 07:08 UTC, Zarick Lau
Details | Diff
The one time memory saving patch (vte-0.11.12-spaces2.patch,7.97 KB, patch)
2005-06-21 07:10 UTC, Zarick Lau
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zarick Lau 2005-06-21 07:03:06 UTC
There are two patches in gnome bugzilla, both are memory usage improvement.
One of it reduce memory footprint for each new terminal,
another patch is a one time saving patching.

You can find detail in here:
http://bugzilla.gnome.org/show_bug.cgi?id=160993

Note, the patch can't cleanly apply on 0.11.11, so I just left it out.
For patch 1, 0.11.13 has included it already

Altogether, I apply both patch against 0.11.12
but only use patch2 for 0.11.13

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Zarick Lau 2005-06-21 07:04:19 UTC
Created attachment 61625 [details]
modified ebuild for 0.11.12
Comment 2 Zarick Lau 2005-06-21 07:04:40 UTC
Created attachment 61626 [details]
Modified ebuild for 0.11.13
Comment 3 Zarick Lau 2005-06-21 07:08:05 UTC
Created attachment 61627 [details, diff]
Patch 1 (save space for each new terminal)

The source:
http://bugzilla.gnome.org/attachment.cgi?id=38525&action=view
Comment 4 Zarick Lau 2005-06-21 07:10:18 UTC
Created attachment 61628 [details, diff]
The one time memory saving patch

The source:
http://bugzilla.gnome.org/attachment.cgi?id=38337&action=view
Comment 5 Leonardo Boshell (RETIRED) gentoo-dev 2005-06-27 09:42:47 UTC
Added the second patch to vte-0.11.13-r1. I'm not sure if we should patch
0.11.12, since the ebuild for vte-0.11.13-r1 would have the same priority as an
ebuild for vte-0.11.12-r1 to become stable in the future, so there isn't much to
gain from it.

Thanks for the report.