patchThe eXtended osip library - Patches: patch #6970, fix building tools/sip_reg on...

 
 

You are not allowed to post comments on this tracker with your current authentification level.

patch #6970: fix building tools/sip_reg on no-mmu/Linux systems

Submitted by:  Mike Frysinger <vapier>
Submitted on:  Fri 06 Nov 2009 11:24:08 AM UTC  
 
Category: NonePriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: Aymeric MOIZARD <aymeric>Open/Closed: Closed

Tue 17 Dec 2013 12:50:50 PM UTC, comment #5:

Hi Again,

configure.in has been renamed to configure.ac for both osip2 & exosip2!

Tks again
Aymeric

Aymeric MOIZARD <aymeric>
Project AdministratorIn charge of this item.
Mon 16 Dec 2013 05:31:51 PM UTC, comment #4:

thanks ;)

if someone does find a system that has fork() but not daemon(), then it'd be easy to add a configure.in test to see if the symbol exists.

on a semi related note, could you rename configure.in to configure.ac ? newer autotools have started warning when people use the old deprecated name.

Mike Frysinger <vapier>
Mon 16 Dec 2013 08:58:35 AM UTC, comment #3:

Hi Mike!

Since you seems to care about this change, even after
a few years, I'll do!

http://git.savannah.gnu.org/cgit/exosip.git/commit/?id=e5889e34822f5311a3a04f099502468a431759fe

Tks for your help.
Regards,
Aymeric

Aymeric MOIZARD <aymeric>
Project AdministratorIn charge of this item.
Mon 16 Dec 2013 05:16:35 AM UTC, comment #2:

like where ? daemon() has been around since 4.4BSD.

Mike Frysinger <vapier>
Fri 13 Dec 2013 08:55:33 AM UTC, comment #1:

Tks for report and sorry for not replying.

Closing now. The patch seems fine, but I guess daemon() might be missing on some platform where fork is availble...

It only concern the test tool, so I leave it as it is!
Aymeric

Aymeric MOIZARD <aymeric>
Project AdministratorIn charge of this item.
Fri 06 Nov 2009 11:24:08 AM UTC, original submission:

since fork() wont work on no-mmu/Linux systems, and the sip_reg code is basically replicating daemon(), use daemon() instead

Mike Frysinger <vapier>

 

Attached Files
file #19001:  libeXosip-daemon.patch added by vapier (516B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by aymeric (Posted a comment)
  • -unavailable- added by vapier (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 16 Dec 2013 08:58:35 AM UTCaymericStatusNone=>Done
    Fri 13 Dec 2013 08:55:33 AM UTCaymericAssigned toNone=>aymeric
      Open/ClosedOpen=>Closed
    Fri 06 Nov 2009 11:24:08 AM UTCvapierAttached File-=>Added libeXosip-daemon.patch, #19001

    Back to the top


    Powered by Savane 3.1-cleanup