Re: request tmpfile(3) wrapping in Guile 1.9 libguile

[Thien-Thi Nguyen]

() Andy Wingo <address@hidden>
() Wed, 10 Feb 2010 00:17:30 +0100

   Yes, I think that's probably the right answer.

On the (faint) suspicion that `scm_fdes_to_port' does not DTRT, given
the file descriptor extracted via `fileno' from the `FILE *', i ran a
small test:

 (write (getpid)) (newline)
 (define tmp (tmpfile))
 (sleep 10)
 (set! tmp #f) ; lose ref
 (let loop () (gc) (sleep 1) (loop))

and monitored using "watch lsof -p PID".  I observed that the descriptor
is listed as "/tmp/FILENAME (deleted)" from the beginning and that after
some time, this entry goes away.  So my trust in `scm_fdes_to_port' is
not broken (cool).  Still, i wonder what others see.

   Yes, this sounds better than mkstemp!. Since you seem to have to code
   already, can you submit two patches with mkstemp and tempfile,
   respectively?

I think you mean `tmpfile'.  Please see attached patch.

   While we're here... Ludovic, Neil and I talked over mail about your
   request to (re)join the Guile project, and we're very happy to have
   your skills and energy.  The one concern that we have is that you
   tend to be a bit cavalier; so until we feel like we're on the same
   page, we would like for you to refrain from committing to master or
   the 1.8 branch.

Thanks for the frank assessment.  I will try to be less cavalier (or
perhaps more "anti-cavalier", once i figure out what that means in this
context).  I appreciate any tips on this aspect.

   Instead, please feel free to (commit to | create) other branches in
   the repository, preferably prefixing those branches with "wip-" if
   you intend them to be rebased at some point.  Please mail the list
   when you would like for us to review and merge your patches into
   master or 1.8.

OK.  How do you feel about a branches named ttn/wip-TOPIC ?

   I recognize that it's a somewhat awkward situation, but we're so
   unfamiliar with each other at this point that a slow rapprochement
   would probably be best to avoid unexpected conflicts.

Agreed.

   That said, welcome back to Guile!

Thanks for giving me another chance.  Perhaps i can avoid squandering
the privelege as i did the previous time.

thi


____________________________________________________

0001-Add-tmpfile-3-to-libguile.patch
Description: Text Data