On October 18, 2006 at 1:52AM +0900,
nori1 (at dolphin.c.u-tokyo.ac.jp) wrote:

> --- debian/rules      (revision 65)
> +++ debian/rules      (revision 68)
> @@ -12,3 +12,24 @@
>  clean::
>       rm -f *.cdb
>       rm -f *.tmp # cdb's intermediate products
> +
> +package := skkdic
> +date := $(shell date '+%Y%m%d')
> +orig_source_dir := ${package}-${date}.orig
> +orig_source_tarball := ${package}_${date}.orig.tar.gz
> +cvs_repos := :pserver:[EMAIL PROTECTED]:/circus/cvsroot
> +cvs_path := skk/dic

I've noticed that the `skk' directory is used in the current
directory, which easily conflicts with the existent directory.

Please consider using more unique name as a temporary directory,
such as `skkdic_YYYYMMDD.tmp'.  (I feel mktemp or similar
isn't needed in this case.  Users should only give care to
the names `skkdic_YYYYMMDD.*'.)

> +clean-orig-source:
> +     rm -rf ${cvs_top} ${orig_source_dir}
> +     rm -f ${orig_source_tarball}

This target is unnecessary, and is unusable on or after next day.
You can remove this target.

--
Tatsuya Kinoshita

Attachment: pgpcSpNB9762E.pgp
Description: PGP signature

Reply via email to