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 authentication level.

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

Submitter:  Mike Frysinger <vapier>
Submitted:  Fri 06 Nov 2009 11:24:08 AM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  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>
Group administrator
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>
Group administrator
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>
Group administrator
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>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

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
  • -email is unavailable- added by aymeric (Posted a comment)
  • -email is unavailable- added by vapier (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-12-16 aymeric StatusNone Done
    2013-12-13 aymeric Assigned toNone aymeric
        Open/ClosedOpen Closed
    2009-11-06 vapier Attached File- Added libeXosip-daemon.patch, #19001

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code