Philip Martin <email@example.com> wrote on 03/14/2006 02:31:37 PM:
> Paul Burba <firstname.lastname@example.org> writes:
> >> > #ifdef AS400
> >> > #define SVN_UTF_UTOE_XLATE_HANDLE "svn-utf-utoe-xlate-handle"
> >> > +#define SVN_UTF_ETOU_XLATE_HANDLE "svn-utf-etou-xlate-handle"
> >> We already have one of those in libsvn_subr/cmdline.c
> > Yes, but how does that help me in io.c? Unless you mean these should
> > made public...should svn_utf.h have all of these?
> > #define SVN_UTF_NTOU_XLATE_HANDLE "svn-utf-ntou-xlate-handle"
> > #define SVN_UTF_UTON_XLATE_HANDLE "svn-utf-uton-xlate-handle"
> No, those are only used in one file.
> > #ifdef AS400
> > #define SVN_UTF_UTOE_XLATE_HANDLE "svn-utf-utoe-xlate-handle"
> > #define SVN_UTF_ETOU_XLATE_HANDLE "svn-utf-etou-xlate-handle"
> > #endif
> I'd suggest putting SVN_UTF_ETOU_XLATE_HANDLE into a libsvn_subr
> header, it doesn't need to be visible outside libsvn_subr.
Thanks, I understand now. Though for approach #3, both
SVN_UTF_UTOE_XLATE_HANDLE and SVN_UTF_ETOU_XLATE_HANDLE are now required
by libsvn_repos. So I placed these in svn_utf.h, is this the correct
> >> I feel a bit guilty about suggesting that since I seem to be asking
> >> for all your patches to be re-written :-/
> > Heh, I'm not worried about that, I just want it done correctly. I see
> > possible options to solve this:
> > 1. Tweak the existing patch to avoid the deadlock issue and call it a
> > 2. Figure out how to make apr_file_t and spawn() work together.
> > 3. Move the patch entirely within hooks.c. Just add a second
> > for run_hook_cmd() along with the existing implementation within a
> > AS400...#else...#endif block.
> > FWIW I like option 3, it is certainly the smallest patch and
> > new APIs.
> 3 is fine as far as I am concerned.
Julian Foad <email@example.com> wrote on 03/14/2006 12:47:53 PM:
> I'd be happy with whatever minimises the impact of these work-arounds on
> code base. (Not just for OS400 but any work-arounds for any stuff that
> ought to handle.)
This patch implements option 3. I'm not sure if anyone noticed this or
not, but the previous patch didn't handle stdin to a hook. This was an
oversight on my part, the attached patch handles this now.
Thanks to both of you for the reviews, please let me know what you think
of this new patch.
This is one of several patches to allow Subversion to run on IBM's
OS400 V5R4. It provides a workaround for various limitations with
IBM's implementation of APR processes.
(SVN_UTF_UTOE_XLATE_HANDLE, SVN_UTF_ETOU_XLATE_HANDLE): New public
xlate keys for EBCDIC <--> UTF-8 conversions.
Include spawn.h and fcntl.h
(run_hook_cmd): New "APR-free" implementation for OS400.
Scanned for SoftLanding Systems, Inc. and SoftLanding Europe Plc by IBM Email Security Management Services powered by MessageLabs.
Received on Fri Mar 17 22:06:26 2006
To unsubscribe, e-mail: firstname.lastname@example.org
For additional commands, e-mail: email@example.com