-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

OK.

I finally got around to refactoring jde-ant-build so that it accepts params via
a *real* (interactive) clause.

This really doesn't add much functionality but I think it is pretty necessary
to move on.  My notes will explain this more.

;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
;;; jde-ant 1.4 notes
;; 
;;; Fri Jan 11 2002 03:59 PM ([EMAIL PROTECTED]): 
;;
;; It has become frustrating using (jde-ant-build) without true interactive
;; arguments.  I set out to refactor jde-ant.el so that jde-ant-build can use
;; true arguments like (jde-ant-build "/project/test/build.xml" "compile") like
;; a real lisp function.  This is important because it allows us to use
;; jde-ant-build more like an API function and we can also use
;; repeat-complex-command like normal Emacs code.
;;
;; This was also a design that could have caused problems in the long term.  For
;; example jde-ant.el depended on function side effects for operation.  For
;; example jde-ant-get-target-alist used jde-ant-find-build-file (in some
;; situations).   
;;
;; Also, looking at the code, I think this refactor will avoid a lot of long
;; term bugs we probably would have ran into.  A good example is that we were
;; using different algorithms in different places for guessing the buildfile.
;; 

;; Changes:
;;
;; - jde-build-ant-command now takes an optional argument buildfile which, if
;;   omitted
;;
;; - The jde-ant-buildfile variable is now deprecated as anything but providing
;;   a default
;;
;; - jde-ant-projecthelp now uses an interactive prompt and also shares a
;;   function with jde-ant-build for reading the buildfile.
;;
;; - jde-ant-find-build-file uses jde-ant-buildfile as a default reference and
;;   returns a string for the buildfile to use or nil.
;;
;; - jde-ant-get-target-alist now accepts a buildfile instead of using find.  We
;; now parameterize the filename.
;;
;; - jde-ant-interactive-buildfile is saved for every use of jde-ant-build and
;; used as the default next time we enter.
;;
;; - We no longer use the Ant version of -find.  We now have a lisp version of
;; this.  This was necessary in order to support jde-ant-read-target and since
;; we already have an implementation it doesn't make sense to have Ant find the
;; file a second time.  
;;
;; - documentation updates.
;;
;; - history variables now just use -history instead arg-history... less confusing.
;;
;; - jde-ant-interactive-target is now gone, this is no longer needed.
;; - jde-ant-interactive-args is now gone, this is no longer needed.

;; - we should be using the -history variables instead of 'interactive'
;; variables.

;; TODO:
;; 
;; - target history should be specific to a buildfile.  If it is not we could
;; accidentally use an invalid target on the wrong file.
;;
;; - maybe jde-ant-read-buildfile and jde-ant-enable-find DON'T HAVE TO BE
;; INCOMPATIBLE.  Maybe if jde-ant-read-buildfile is enabled we could use it as
;; the default.  That would work nicely.

;;; TESTS:
;;
;; We need to test that the following functionality still works.
;;
;; - (CONFIRMED) jde-ant-read-buildfile
;; - (CONFIRMED) jde-ant-read-target
;; - (CONFIRMED) jde-ant-enable-find
;; - (CONFIRMED) jde-ant-complete-target
;; 
;; - (CONFIRMED) last target is used if we execute this interactively.
;;
;; - (CONFIRMED) when we are interactively prompting for a filename using 
read-file-name,
;;   use the last selected file by default.  When we first enter we should have
;;   NO filename except the default directory.

;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;

Attachment: jde-ant.el
Description: application/emacs-lisp


- -- 
Kevin A. Burton ( [EMAIL PROTECTED], [EMAIL PROTECTED], [EMAIL PROTECTED] )
             Location - San Francisco, CA, Cell - 415.595.9965
        Jabber - [EMAIL PROTECTED],  Web - http://relativity.yi.org/

YEAH!!! I'M A MIME!

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: Get my public key at: http://relativity.yi.org/pgpkey.txt

iD8DBQE8QQ0+AwM6xb2dfE0RAoBwAJ44JCdQPj14lPv27KMNVg5v0pIOqACfSY+6
j/8uJ8BeAjqBNnhkK622NTs=
=OEer
-----END PGP SIGNATURE-----

Reply via email to