Mailing-List: contact bloodhound-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: bloodhound-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of matevzb@gmail.com designates
 209.85.219.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAGMZAuMQ8axzEze0edYMhi7r=TacdbqLPH6ZBWMnh+rxmPGMWA@mail.gmail.com>
References: 
 <CA+DjKZc4rh9Fd6Th=kvVQMD3qpnZ+cyjh=K-EHwjo0zcv_P3TQ@mail.gmail.com>
 <CAJSKAsjgAQMwMNFwO2CNyTR71mLpk0rVmR44OdVcfCa8aBztWg@mail.gmail.com>
 <2BA32F18-E010-433A-8E9F-359192ABA8F8@wandisco.com>
 <CA+DjKZcBwnEJfn5VLdET79WF2buQu1r_xcHVcJp1h=2KFefFgA@mail.gmail.com>
 <CA+DjKZce2r0Ke7BRutgtebAT-sNj=9XMAiPnbmTX8fvAaMDyoQ@mail.gmail.com>
 <CAGMZAuPd49K=QRfb=PbEHLO_rTY=ab85AcuGjnS_ubxx6xKa2A@mail.gmail.com>
 <CAGMZAuPsme2g_WPJ_Rn_kyh1p35T_+Zx5Y2mFnVz70L293Hmcg@mail.gmail.com>
 <CAGMZAuPkL3PKLLY1TiKR22uYK+Jwp4zrECHM8zgUnEDyBXoL-A@mail.gmail.com>
 <CA+DjKZeQEj+x9usbu0iSCcTPUqfJ81xRKBjaMgr9fzsxcEKidw@mail.gmail.com>
 <CAGMZAuOaDBHox89GoNjnpQtcXeH6io_nFOMUHc2GmEfzeM6oJw@mail.gmail.com>
 <CA+DjKZcMyg0g4bjJ+FVt94i_H49_VoR7NpwtMCxqYid3=VoALg@mail.gmail.com>
 <CAGMZAuMQ8axzEze0edYMhi7r=TacdbqLPH6ZBWMnh+rxmPGMWA@mail.gmail.com>
From: Matevz Bradac <matevzb@gmail.com>
Date: Mon, 3 Dec 2012 09:25:58 +0100
Message-ID: 
 <CA+DjKZdE8KP1YbFJKuUiBBjzXc=EEnWQDwgy9UM=M6KvZAXdNw@mail.gmail.com>
Subject: Re: Genshi XHTML + HTML 5
To: bloodhound-dev@incubator.apache.org
Content-Type: text/plain; charset=UTF-8

Hi,

Great, thanks for all the suggestions. =)
I'll start working on trac changes then, it would make more sense to
make trac itself more configurable
than to create workarounds within bloodhound (and possibly other
projects based on trac).

--
matevz

On Mon, Dec 3, 2012 at 12:52 AM, Olemis Lang <olemis@gmail.com> wrote:
> Hi !
>
> On 12/2/12, Matevz Bradac <matevzb@gmail.com> wrote:
>> I tried solving this on the stream level, but no luck so far.
>
> However you have been doing an extraordinary research here . Good work .
> :)
>
> [...]
>>
>> Due to the above complications (and those not yet discovered), I'd say
>> it would be easier and less error-prone
>> to just override the genshi.output.get_serializer() with BH's wrapper.
>> The function is pretty trivial and unlikely
>> to change much, so it looks like a safe alternative.
>>
>
> I'm not very aware of the details . Nonetheless I'd just say for the
> moment that there's more chance to accept a patch (i.e. not
> monkeypatching ;) for Chrome.render_template than another for Genshi .
> In the end we are already distributing a modified copy of Trac , so
> patches of this kind are welcome , especially if there's a good reason
> to do so . So at this point don't be afraid of proposing a refactoring
> for that method in order to be able to choose another doctype . Just
> try to keep it as compatible with previous solution as possible ,
> since there is no need to break Trac if e.g. Bloodhound theme is
> disabled
>
> OTOH creating another vendor branch for Genshi is quite unlikely to
> happen soon .
>
> Keep up the good work !
> ;)
>
> --
> Regards,
>
> Olemis.
>
> Blog ES: http://simelo-es.blogspot.com/
> Blog EN: http://simelo-en.blogspot.com/
>
> Featured article: