[haiku-development] Re: Mercurial - ifdef __BEOS__ question

  • From: "scott mc" <scottmc2@xxxxxxxxx>
  • To: haiku-development@xxxxxxxxxxxxx
  • Date: Tue, 9 Sep 2008 16:41:30 -0700

On Tue, Sep 9, 2008 at 4:35 PM, François Revol <revol@xxxxxxx> wrote:
>> While checking into porting Mercurial I ran into an ifdef __BEOS__ to
>
> Hmm mercurial shouldn't need any porting, it's a python script.
> It actually runs fine on BeOS (I needed it for XEmacs and OSS).
> You probably talk about the bdiff and mpatch tools that go with it...
>
>> include <ByteOrder.h> and else include <arpa/inet.h>
>> Seems Haiku has both, so which one should be used?  Since __BEOS__ is
>> currently defined in Haiku it will of course use that one if no
>> changes are made.
>> Files in question are bdiff.c and mpatch.c in the mercurial folder.
>
> I think it should be ok.
> It's likely only needed for net_server, as BONE has arpa/inet.h.
> But it shouldn't harm as we still have ByteOrder.h anyway, does it
> still build ?
>
> François.
>
>

It's got some minor issues.  Such as using Python2.4 for BeOS to build
it... that causes it to try to use -lnet instead of -lnetwork, easy to
fix, fix it in my python.  Next is that it tries to install to
/usr/local which I haven't dealt with yet.  I just posted a quick
patch for the cases listed above.  I might look into the rest soon.
Thanks for the quick replies.
-scott

Other related posts: