{"thread":{"id":"8589","subject":"[PATCH] Add a guilt-export(1) command to export a guilt series to quilt.","startedAt":"2007-06-13T14:01:53Z","lastAt":"2007-06-14T10:38:32Z","messageCount":6,"participants":["Pierre Habouzit","Benjamin Sergeant","Josef Sipek"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"44931","messageId":"1181743313509-git-send-email-madcoder@debian.org","threadId":"8589","inReplyTo":null,"subject":"[PATCH] Add a guilt-export(1) command to export a guilt series to quilt.","fromName":"Pierre Habouzit","fromEmail":"madcoder@debian.org","sentAt":"2007-06-13T14:01:53Z","receivedAt":"2007-06-13T14:01:53Z","isPatch":true,"sender":{"key":"madcoder@debian.org","avatar":"https://avatars.githubusercontent.com/u/44708?v=4"},"body":"Signed-off-by: Pierre Habouzit \n---\n Documentation/guilt-export.txt | 30 ++++++++++++++++++++++++++++++\n guilt-export | 26 ++++++++++++++++++++++++++\n 2 files changed, 56 insertions(+), 0 deletions(-)\n create mode 100644 Documentation/guilt-export.txt\n create mode 100755 guilt-export\n\ndiff --git a/Documentation/guilt-export.txt b/Documentation/guilt-export.txt\nnew file mode 100644\nindex 0000000..b7b0a4b\n--- /dev/null\n+++ b/Documentation/guilt-export.txt\n@@ -0,0 +1,30 @@\n+guilt-export(1)\n+===============\n+\n+NAME\n+----\n+guilt-export - Export a patch series (to be used by quilt)\n+\n+SYNOPSIS\n+--------\n+include::usage-guilt-export.txt[]\n+\n+DESCRIPTION\n+-----------\n+Export a guilt series to be used by quilt.\n+\n+OPTIONS\n+-------\n+::\n+ Name of the directory to export the patch series to.\n+ (defaults to patches).\n+\n+Author\n+------\n+Written by Pierre Habouzit \n+\n+Documentation\n+-------------\n+Documentation by Pierre Habouzit \n+\n+include::footer.txt[]\ndiff --git a/guilt-export b/guilt-export\nnew file mode 100755\nindex 0000000..9ff9924\n--- /dev/null\n+++ b/guilt-export\n@@ -0,0 +1,26 @@\n+#!/bin/bash\n+#\n+# Copyright (c) Pierre Habouzit, 2007\n+#\n+\n+USAGE=\"[]\"\n+. guilt\n+\n+if [ $# -gt 1 ]; then\n+\tusage\n+fi\n+target_dir=${1:-\"patches\"}\n+\n+if [ -e \"$target_dir\" ]; then\n+\tdie \"Specified directory already exists\"\n+fi\n+\n+trap \"rm -rf \\\"$target_dir\\\"\" 0\n+mkdir -p \"$target_dir\"\n+\n+get_series | tee \"$target_dir/series\" | while read p; do\n+\tcp \"$GUILT_DIR/$branch/$p\" \"$target_dir/$p\"\n+done\n+\n+trap - 0\n+echo \"Series exported to \\\"$target_dir\\\" sucessfully.\"\n-- \n1.5.2.1\n"},{"id":"44946","messageId":"1621f9fa0706130810n17bc3f67h1c539d75b3aed4ed@mail.gmail.com","threadId":"8589","inReplyTo":"1181743313509-git-send-email-madcoder@debian.org","subject":"Re: [PATCH] Add a guilt-export(1) command to export a guilt series to quilt.","fromName":"Benjamin Sergeant","fromEmail":"bsergean@gmail.com","sentAt":"2007-06-13T15:10:36Z","receivedAt":"2007-06-13T15:10:36Z","isPatch":true,"sender":{"key":"bsergean@gmail.com","avatar":null},"body":"For the shebang:\n#!/usr/bin/env bash\nshould works well and should work on non-linux boxes that did not\nadded a link in /bin/bash.\n\nMy 2 cents.\n\nOn 6/13/07, Pierre Habouzit wrote:\n> Signed-off-by: Pierre Habouzit \n> ---\n> Documentation/guilt-export.txt | 30 ++++++++++++++++++++++++++++++\n> guilt-export | 26 ++++++++++++++++++++++++++\n> 2 files changed, 56 insertions(+), 0 deletions(-)\n> create mode 100644 Documentation/guilt-export.txt\n> create mode 100755 guilt-export\n>\n> diff --git a/Documentation/guilt-export.txt b/Documentation/guilt-export.txt\n> new file mode 100644\n> index 0000000..b7b0a4b\n> --- /dev/null\n> +++ b/Documentation/guilt-export.txt\n> @@ -0,0 +1,30 @@\n> +guilt-export(1)\n> +===============\n> +\n> +NAME\n> +----\n> +guilt-export - Export a patch series (to be used by quilt)\n> +\n> +SYNOPSIS\n> +--------\n> +include::usage-guilt-export.txt[]\n> +\n> +DESCRIPTION\n> +-----------\n> +Export a guilt series to be used by quilt.\n> +\n> +OPTIONS\n> +-------\n> +::\n> + Name of the directory to export the patch series to.\n> + (defaults to patches).\n> +\n> +Author\n> +------\n> +Written by Pierre Habouzit \n> +\n> +Documentation\n> +-------------\n> +Documentation by Pierre Habouzit \n> +\n> +include::footer.txt[]\n> diff --git a/guilt-export b/guilt-export\n> new file mode 100755\n> index 0000000..9ff9924\n> --- /dev/null\n> +++ b/guilt-export\n> @@ -0,0 +1,26 @@\n> +#!/bin/bash\n> +#\n> +# Copyright (c) Pierre Habouzit, 2007\n> +#\n> +\n> +USAGE=\"[]\"\n> +. guilt\n> +\n> +if [ $# -gt 1 ]; then\n> + usage\n> +fi\n> +target_dir=${1:-\"patches\"}\n> +\n> +if [ -e \"$target_dir\" ]; then\n> + die \"Specified directory already exists\"\n> +fi\n> +\n> +trap \"rm -rf \\\"$target_dir\\\"\" 0\n> +mkdir -p \"$target_dir\"\n> +\n> +get_series | tee \"$target_dir/series\" | while read p; do\n> + cp \"$GUILT_DIR/$branch/$p\" \"$target_dir/$p\"\n> +done\n> +\n> +trap - 0\n> +echo \"Series exported to \\\"$target_dir\\\" sucessfully.\"\n> --\n> 1.5.2.1\n>\n> -\n> To unsubscribe from this list: send the line \"unsubscribe git\" in\n> the body of a message to majordomo@vger.kernel.org\n> More majordomo info at http://vger.kernel.org/majordomo-info.html\n>\n"},{"id":"44948","messageId":"20070613151640.GI5311@artemis.intersec.eu","threadId":"8589","inReplyTo":"1621f9fa0706130810n17bc3f67h1c539d75b3aed4ed@mail.gmail.com","subject":"Re: [PATCH] Add a guilt-export(1) command to export a guilt series to quilt.","fromName":"Pierre Habouzit","fromEmail":"madcoder@debian.org","sentAt":"2007-06-13T15:16:40Z","receivedAt":"2007-06-13T15:16:40Z","isPatch":true,"sender":{"key":"madcoder@debian.org","avatar":"https://avatars.githubusercontent.com/u/44708?v=4"},"body":"On Wed, Jun 13, 2007 at 08:10:36AM -0700, Benjamin Sergeant wrote:\n> For the shebang:\n> #!/usr/bin/env bash\n> should works well and should work on non-linux boxes that did not\n> added a link in /bin/bash.\n\n FWIW it would spawn a process for nothin, and every other guilt\ncommand uses that. People using it on non linux box would have to run a\nsed on every shebang at install time, that's not _that_ hard anyways,\nand is common practice with python or perl scripts e.g. IMHO it's not a\nproblem.\n\n-- \n·O· Pierre Habouzit\n··O madcoder@debian.org\nOOO http://www.madism.org\n"},{"id":"44952","messageId":"20070613155104.GB18989@filer.fsl.cs.sunysb.edu","threadId":"8589","inReplyTo":"1181743313509-git-send-email-madcoder@debian.org","subject":"Re: [PATCH] Add a guilt-export(1) command to export a guilt series to quilt.","fromName":"Josef Sipek","fromEmail":"jsipek@fsl.cs.sunysb.edu","sentAt":"2007-06-13T15:51:04Z","receivedAt":"2007-06-13T15:51:04Z","isPatch":true,"sender":{"key":"jsipek@fsl.cs.sunysb.edu","avatar":null},"body":"On Wed, Jun 13, 2007 at 04:01:53PM +0200, Pierre Habouzit wrote:\n> Signed-off-by: Pierre Habouzit \n> ---\n> Documentation/guilt-export.txt | 30 ++++++++++++++++++++++++++++++\n> guilt-export | 26 ++++++++++++++++++++++++++\n> 2 files changed, 56 insertions(+), 0 deletions(-)\n> create mode 100644 Documentation/guilt-export.txt\n> create mode 100755 guilt-export\n\nApplied.\n\nThanks.\n\nJosef \"Jeff\" Sipek.\n\n-- \nComputer Science is no more about computers than astronomy is about\ntelescopes.\n\t\t- Edsger Dijkstra\n"},{"id":"44953","messageId":"20070613155454.GC18989@filer.fsl.cs.sunysb.edu","threadId":"8589","inReplyTo":"1621f9fa0706130810n17bc3f67h1c539d75b3aed4ed@mail.gmail.com","subject":"Re: [PATCH] Add a guilt-export(1) command to export a guilt series to quilt.","fromName":"Josef Sipek","fromEmail":"jsipek@fsl.cs.sunysb.edu","sentAt":"2007-06-13T15:54:54Z","receivedAt":"2007-06-13T15:54:54Z","isPatch":true,"sender":{"key":"jsipek@fsl.cs.sunysb.edu","avatar":null},"body":"On Wed, Jun 13, 2007 at 08:10:36AM -0700, Benjamin Sergeant wrote:\n> For the shebang:\n> #!/usr/bin/env bash\n> should works well and should work on non-linux boxes that did not\n> added a link in /bin/bash.\n \nIdeally, the best thing to do is to change guilt to use /bin/sh. Right now,\nthere are a number of bash-isms that are used. And if a system doesn't have\n/bin/sh, it doesn't deserve to run any useful software ;)\n\nPatches are welcomed ;)\n\nJosef \"Jeff\" Sipek.\n\n-- \nReality is merely an illusion, albeit a very persistent one.\n\t\t- Albert Einstein\n"},{"id":"45060","messageId":"20070614103832.GA23324@artemis","threadId":"8589","inReplyTo":"20070613155454.GC18989@filer.fsl.cs.sunysb.edu","subject":"Re: [PATCH] Add a guilt-export(1) command to export a guilt series to quilt.","fromName":"Pierre Habouzit","fromEmail":"madcoder@debian.org","sentAt":"2007-06-14T10:38:32Z","receivedAt":"2007-06-14T10:38:32Z","isPatch":true,"sender":{"key":"madcoder@debian.org","avatar":"https://avatars.githubusercontent.com/u/44708?v=4"},"body":"On Wed, Jun 13, 2007 at 11:54:54AM -0400, Josef Sipek wrote:\n> On Wed, Jun 13, 2007 at 08:10:36AM -0700, Benjamin Sergeant wrote:\n> > For the shebang:\n> > #!/usr/bin/env bash\n> > should works well and should work on non-linux boxes that did not\n> > added a link in /bin/bash.\n> \n> Ideally, the best thing to do is to change guilt to use /bin/sh. Right now,\n> there are a number of bash-isms that are used. And if a system doesn't have\n> /bin/sh, it doesn't deserve to run any useful software ;)\n> \n> Patches are welcomed ;)\n\n Makes sense. I'm currently working on it (so that nobody loses time on\nthat issue).\n\n I'm mostly done with guilt(1), I still need to take some \"local\"\nvariables down (especially in guilt_push or whatever the function name\nis). Most of the rest has been dealt with already.\n\n\n I should send a patch serie towards POSIX-shell-isation soon, as soon\nas I've tested the patches with bash, zsh, dash and posh.\n\n Cheers,\n-- \n·O· Pierre Habouzit\n··O madcoder@debian.org\nOOO http://www.madism.org\n"}]}