PiRSquared17 / svnplot

Automatically exported from code.google.com/p/svnplot
0 stars 0 forks source link

Windows-specific conventions, does not run out-of-the-box on Unix #83

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
## What steps will reproduce the problem?

1. log in on a Linux machine
2. wget http://svnplot.googlecode.com/files/SVNPlot-0.7.6.zip ; unzip 
SVNPlot-0.7.6.zip ; cd SVNPlot-0.7.6/svnplot ; chmod a+x svnlog2sqlite.py
3. ./svnlog2sqlite.py a b

What is the expected output? What do you see instead?

## Observed output

$ ./svnlog2sqlite.py a b
./svnlog2sqlite.py: ligne 13:
svnlog2sqlite.py
Copyright (C) 2009 Nitin Bhide (nitinbhide@gmail.com)

This module is part of SVNPlot (http://code.google.com/p/svnplot) and is 
released under
the New BSD License: http://www.opensource.org/licenses/bsd-license.php
--------------------------------------------------------------------------------
------

python script to convert the Subversion log into an sqlite database
The idea is to use the generated SQLite database as input to Matplot lib for
creating various graphs and analysis. The graphs are inspired from graphs
generated by StatSVN/StatCVS.
: Aucun fichier ou dossier de ce type
./svnlog2sqlite.py: ligne 14: $'\r' : commande introuvable

The mouse cursor changes into a crosshair until I click, several times.

## Expected output

Updating the subversion log
Repository : a
SVN Log database filepath : b
Extract Changed Line Count : False
        please use -l option. if you want to extract linecount information.

What version of the product are you using? On what operating system?

SVNPlot-0.7.6.zip on Ubuntu 12.04

## Please provide any additional information below.

Reason is easy: file does not start with

#!/usr/bin/env python

Since python programs are usually highly portable, it makes sense to always 
obey both Windows (name them *.py) and Unix (start with #!/usr/bin/env python, 
linefeed without CR) conventions.

## Once these are added, it seems to work.
## One can continue following the README and import data.

References :

http://en.wikipedia.org/wiki/Shebang_%28Unix%29
http://en.wikipedia.org/wiki/Carriage_return

I'm willing to offer you a patch for easy inclusion in your source code.

Original issue reported on code.google.com by stephane...@gmail.com on 11 May 2012 at 3:45

GoogleCodeExporter commented 9 years ago
Hi stephane
first sorry for the delayed reply. I will appreciate the patch. I will include 
it in the next release.

Original comment by nitinbh...@gmail.com on 4 Aug 2012 at 7:18

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r549.

Original comment by nitinbh...@gmail.com on 4 Aug 2012 at 12:46