** Changed in: dput (Ubuntu)
Importance: Undecided => Low
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1762572
Title:
regression: sftp method no longer uses temporary file name during
upload
Thanks for the patch! I think Python has built-in functions for
generating files names we could use, tough, that might look nicer.
I'll have a look later. I also need to update my git Branch for
upstreaming.
** Changed in: dput (Ubuntu)
Status: Confirmed => Triaged
--
You received this b
The attachment "0001-fix-atomic-upload-regression-for-sftp-method.patch"
seems to be a patch. If it isn't, please remove the "patch" flag from
the attachment, remove the "patch" tag, and if you are a member of the
~ubuntu-reviewers, unsubscribe the team.
[This is an automated message performed by
It is a feature to me. I can't think of an easy way to work around it
otherwise.
Using a temporary file like this seems common. In fact dput used to do
this, which is why I consider it a regression.
Similar advice here: https://stackoverflow.com/questions/25119076/how-
to-do-an-atomic-sftp-file-t
Thank you for your bug report, is the use of the temporary filename
really a feature or is that an implementation detail you happened to
rely on?
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1762572
Here is a patch that I have been using for a while.
** Patch added: "0001-fix-atomic-upload-regression-for-sftp-method.patch"
https://bugs.launchpad.net/ubuntu/+source/dput/+bug/1762572/+attachment/5235354/+files/0001-fix-atomic-upload-regression-for-sftp-method.patch
--
You received this bu
Status changed to 'Confirmed' because the bug affects multiple users.
** Changed in: dput (Ubuntu)
Status: New => Confirmed
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1762572
Title:
regre
It looks like this can be fixed by copying (parts of) the _put() method
from bzrlib
def _put(self, abspath, f, mode=None):
"""Helper function so both put() and copy_abspaths can reuse the code"""
tmp_abspath = '%s.tmp.%.9f.%d.%d' % (abspath, time.time(),