diff options
author | Kaz Kylheku <kaz@kylheku.com> | 2021-02-28 09:26:44 -0800 |
---|---|---|
committer | Kaz Kylheku <kaz@kylheku.com> | 2021-02-28 09:26:44 -0800 |
commit | 920782cb237657e912ac154dbc5117b6ac8e1229 (patch) | |
tree | 1804b9a15ea8d36ed543f6d12052796281839d04 | |
parent | 7058d049384311d03836c6f71f225e99ec1ff040 (diff) | |
download | txr-920782cb237657e912ac154dbc5117b6ac8e1229.tar.gz txr-920782cb237657e912ac154dbc5117b6ac8e1229.tar.bz2 txr-920782cb237657e912ac154dbc5117b6ac8e1229.zip |
copy-file: get rid of package.
* share/txr/stdlib/copy-file.tl (package copy-file): Removed.
I cannot remember why I used this, but it was probably because
this source file was developed in "user space". Then it likely
became an experiment in the use of merge-delete-package to
define material in a temporary package which is merged down to
the ultimate target package like sys. Why get rid of this?
It's causing a problem. The second and subsequent recompile of
copy-path-rec causes its internal block not to be optimized
away. I spotted this while checking whether a recompile of the
library using the fully compiled library is different from
the initial compile: copy-path-rec's block is not optimized away.
-rw-r--r-- | share/txr/stdlib/copy-file.tl | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/share/txr/stdlib/copy-file.tl b/share/txr/stdlib/copy-file.tl index 198424af..aaff63f2 100644 --- a/share/txr/stdlib/copy-file.tl +++ b/share/txr/stdlib/copy-file.tl @@ -25,13 +25,7 @@ ;; OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. (eval-only - (defsymacro copy-size 65536) - - (defpackage copy-file - (:fallback usr sys) - (:use-syms usr:perms usr:times usr:owner usr:symlinks)) - - (in-package copy-file)) + (defsymacro copy-size 65536)) (defstruct copy-path-opts () perms times owner symlinks (euid (geteuid))) @@ -218,6 +212,3 @@ (let ((rst (stat ref-path))) (utimes s 0 nil rst.mtime rst.mtime-nsec)) (utimes s 0 nil 0 t)))) - -(eval-only - (merge-delete-package 'sys)) |