Pushed. Thanks!
I've decided not to enhance Dump dirs storage to accept the same directory more than once. It wouldn't have any added value. In most cases users uploads the same error.
Jakub
On Thursday 11 of April 2013 13:16:20 Denys Vlasenko wrote:
Signed-off-by: Denys Vlasenko dvlasenk@redhat.com
src/plugins/reporter-upload.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/plugins/reporter-upload.c b/src/plugins/reporter-upload.c index bff8105..a527431 100644 --- a/src/plugins/reporter-upload.c +++ b/src/plugins/reporter-upload.c @@ -49,7 +49,9 @@ static int create_and_upload_archive( /* Create a child gzip which will compress the data */ /* SELinux guys are not happy with /tmp, using /var/run/abrt */ /* Reverted back to /tmp for ABRT2 */
- tempfile = xasprintf("/tmp/abrt-upload-%s-%lu.tar.gz",
iso_date_string(NULL), (long)getpid()); + tempfile = concat_path_basename("/tmp", dump_dir_name);
- tempfile = append_to_malloced_string(tempfile, ".tar.gz");
- int pipe_from_parent_to_child[2]; xpipe(pipe_from_parent_to_child); child = vfork();