Patchwork Get rid of some code duplication in parsemail-batch.sh.

login
register
mail settings
Submitter Guilherme Salgado
Date Feb. 28, 2011, 11:54 a.m.
Message ID <20110228115401.9401.7518.stgit@localhost6.localdomain6>
Download mbox | patch
Permalink /patch/84822/
State Accepted
Commit c237b832d7bfa9bad2ba76e89888dcbfa4c924dc
Headers show

Comments

Guilherme Salgado - Feb. 28, 2011, 11:54 a.m.
It now uses parsemail.sh instead of parsemail.py as the former sets the
required environment variables that were being set in parsemail-batch.sh.

Signed-off-by: Guilherme Salgado <guilherme.salgado@linaro.org>
---
 apps/patchwork/bin/parsemail-batch.sh |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)
Jeremy Kerr - March 30, 2011, 7:04 a.m.
Hi Guilherme,

> It now uses parsemail.sh instead of parsemail.py as the former sets the
> required environment variables that were being set in parsemail-batch.sh.

Thanks, applied.


Jeremy

Patch

diff --git a/apps/patchwork/bin/parsemail-batch.sh b/apps/patchwork/bin/parsemail-batch.sh
index d786022..31ef4f0 100755
--- a/apps/patchwork/bin/parsemail-batch.sh
+++ b/apps/patchwork/bin/parsemail-batch.sh
@@ -19,7 +19,7 @@ 
 # along with Patchwork; if not, write to the Free Software
 # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
-PATCHWORK_BASE="/srv/patchwork"
+PATCHWORK_BINDIR=`dirname $0`
 
 if [ $# -ne 1 ]
 then
@@ -41,11 +41,5 @@  ls -1rt "$mail_dir" |
 while read line;
 do
 	echo $line
-	PYTHONPATH="$PATCHWORK_BASE/apps":"$PATCHWORK_BASE/lib/python" \
-		DJANGO_SETTINGS_MODULE=settings \
-		"$PATCHWORK_BASE/apps/patchwork/bin/parsemail.py" < \
-		"$mail_dir/$line"
+	$PATCHWORK_BINDIR/parsemail.sh < "$mail_dir/$line"
 done
-
-
-