[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] Remove the useless scripts directory and script under it

On 2012年04月18日 15:12, Guannan Ren wrote:
On 04/17/2012 10:19 PM, Osier Yang wrote:
The script is useful only if all the scripts have good document
comments. Unfortunately, it's not true, and it's burden for
developers to write those documents in specific format somehow,
on the other hand, because we introduced required_opts, optional_opts
for each cases to generate the usage strings, and new option "-t"
to print template for cases, it's no need to write documents
in specific format for scripts now.
scripts/gen_case_doc.sh | 21 ---------------------
1 files changed, 0 insertions(+), 21 deletions(-)
delete mode 100644 scripts/gen_case_doc.sh

diff --git a/scripts/gen_case_doc.sh b/scripts/gen_case_doc.sh
deleted file mode 100644
index e814532..0000000
--- a/scripts/gen_case_doc.sh
+++ /dev/null
@@ -1,21 +0,0 @@
-#! /bin/sh
-# Create documentation of cases in Python
-FILES=`find $PWD/repos/Python -type f -name "*.py"`
-for i in ${FILES}; do
- if [ $(basename $i) != '__init__.py' ]; then
- DIRNAME=$(dirname $i)
- DIRNAME=$(echo $DIRNAME | sed -e 's:repos:doc:g')
- fi
- if ! [ -e $DIRNAME ]; then
- mkdir -p $DIRNAME
- fi
- pydoc -w $i
- cd -
-exit $?

I assume that the folder has been removed together.

Thanks, pushed. The directory can be left, or remove,
either is fine, as there is follow up patch to move
install-autostart-server to scripts.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]