Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 26181 - Patch for bitchx, This patch fix hebrew bidi in bitchx
Summary: Patch for bitchx, This patch fix hebrew bidi in bitchx
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Chuck Short (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-08-08 00:39 UTC by Genady Okrain
Modified: 2003-08-19 19:09 UTC (History)
2 users (show)

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


Attachments
Hebrew bidi support patch (hebrew.patch,477 bytes, patch)
2003-08-08 00:40 UTC, Genady Okrain
Details | Diff
Bitchx hebrew patch (bitchx-1.0.19-hebrew.patch,477 bytes, patch)
2003-08-08 20:34 UTC, Chuck Short (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Genady Okrain 2003-08-08 00:39:29 UTC
This patch enables hebrew bidi in bitchx

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Genady Okrain 2003-08-08 00:40:19 UTC
Created attachment 15733 [details, diff]
Hebrew bidi support patch
Comment 2 Chuck Short (RETIRED) gentoo-dev 2003-08-08 20:32:07 UTC
Comment on attachment 15733 [details, diff]
Hebrew bidi support patch

Patch has been renamed.
Comment 3 Chuck Short (RETIRED) gentoo-dev 2003-08-08 20:34:06 UTC
Created attachment 15784 [details, diff]
Bitchx hebrew patch

The patch has been renamed to co-exist with the naming scheme of the other
bitchx patches.
Comment 4 Chuck Short (RETIRED) gentoo-dev 2003-08-08 20:39:13 UTC
This patch applies cleanly against bitchx-1.0.19-r5.ebuild. I just added the following line to the src_unpack():

epatch ${FILESDIR}/${P}-hebrew.patch || die

I dont think it would hurt to add this patch to the bitchx ebuild. Greg could this patch be included in whith the bitchx ebuild. 

Thanks
chuck
Comment 5 Greg Fitzgerald (RETIRED) gentoo-dev 2003-08-19 19:09:45 UTC
In CVS now.