This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: cygport improvements: upload, fish, src_prep_fini_hook
- From: Andrew Schulman <schulman dot andrew at epa dot gov>
- To: cygwin-apps at cygwin dot com
- Date: Wed, 03 Dec 2014 16:40:57 -0500
- Subject: Re: cygport improvements: upload, fish, src_prep_fini_hook
- Authentication-results: sourceware.org; auth=none
- References: <i1c74ah3hi6gdirp06o45tq2kcstclnr14 at 4ax dot com> <i1c74ah3hi6gdirp06o45tq2kcstclnr14-e09XROE/p8c at public dot gmane dot org> <544D0CC5 dot 9030600 at cygwin dot com> <nu2s4al5tup14gfvge083ri8u9j6t62c7m at 4ax dot com> <nu2s4al5tup14gfvge083ri8u9j6t62c7m-e09XROE/p8c at public dot gmane dot org> <547F5B88 dot 3020403 at cygwin dot com>
> Here's what I have at the moment based on your branch as of a few weeks
> ago. However, with password-protected SSH keys, the password prompt
> isn't handled properly. Any ideas?
OK. Looks good.
Re password prompts: I see the problem. It's because I echo the lftp
script to stdout, and pipe it into lftp -f /dev/stdin. So stdin is
unavailable for the password prompt.
I think using process substitution instead will be an easy fix for this.
I'll code this up, test it and push it out shortly.
BTW I see that you commented out the line:
echo "rm -f !ready || echo -n"
in the lftp script in __pkg_upload(). That line is there to prevent a race
condition if the maintainer has already made one upload, then later starts
to make another one, at the same time that upset starts to copy files from
the upload area. In that case, upset could copy partially uploaded or
corrupt files. To prevent that, we first remove !ready, then put it back
later after the new upload is complete.
Andrew