12:02 p.m.
Hi!
So, I'm back from my vacation and try to get the package stuff up to
date. Now, I try to 'make bindist' in a freshly cloned tree and end up
with the following error:
...
# #### See if we can't CLEAN this up
xemacs -vanilla -batch -eval '(setq stack-trace-on-error t
load-always-display-messages t load-ignore-out-of-date-elc-files t
load-show-full-path-in-messages t)' \
-l /usr/users/nk/pkg-build/test/hack-package-index.el -f batch-hack-package-index \
xemacs-base package-info /usr/users/nk/pkg-build/staging/package-index standard
Wrote /usr/users/nk/pkg-build/staging/package-index
xemacs -vanilla -batch -eval '(setq stack-trace-on-error t
load-always-display-messages t load-ignore-out-of-date-elc-files t
load-show-full-path-in-messages t)' -l
/usr/users/nk/pkg-build/test/package-net-packages.el \
-l /usr/users/nk/pkg-build/staging/package-index \
-f package-net-packages-batch-convert-index-to-ini /usr/users/nk/pkg-build/staging
standard
Wrote /usr/users/nk/pkg-build/staging/setup-packages.ini # bind (
stack-trace-on-signal debug-on-signal stack-trace-on-error debug-on-error)
signal(error ("/usr/users/nk/pkg-build/staging is a directory"))
# bind (data error-symbol)
signal-error(error ("/usr/users/nk/pkg-build/staging is a directory"))
# bind (error-data)
normal-top-level()
# (condition-case ... . error)
# (catch top-level
...)
/usr/users/nk/pkg-build/staging is a directory
xemacs exiting
.
make[2]: *** [setup-info] Error 255
make[2]: Leaving directory `/usr/users/nk/pkg-build/test/xemacs-packages/xemacs-base'
make[1]: *** [xemacs-base/install-for-bindist.target] Error 2
make[1]: Leaving directory `/usr/users/nk/pkg-build/test/xemacs-packages'
make: *** [xemacs-packages/install-for-bindist.target] Error 2
Erm, has this changed in one of the last commits? I've done it this way
since like forever ...
Thanks,
norbert.
--
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-beta
6:05 a.m.
>>>> Norbert Koch <viteno(a)xemacs.org> writes:
Now, I try to 'make bindist' in a freshly cloned tree and end
up
with the following error:
...
[...]
Erm, has this changed in one of the last commits? I've done it
this way
since like forever ...
For what it is worth: I can't reproduce this using 21.5 and a not
completely fresh clone (but make-cleaned before make bindist anyway.)
(Not much help I'm afraid.)
I could add make bindist to the package smoketest if we feel that this
needs to be tested on a regular basis!?
Yours
--
%% Mats
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-beta
10:29 p.m.
* Mats Lidell <matsl(a)xemacs.org>:
For what it is worth: I can't reproduce this using 21.5 and a
not
completely fresh clone (but make-cleaned before make bindist anyway.)
(Not much help I'm afraid.)
Bugger.
I could add make bindist to the package smoketest if we feel that
this
needs to be tested on a regular basis!?
No, I don't think it's worth the effort. I'll recheck my setup and try
to get this up and running again.
norbert.
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-beta
5:51 p.m.
On Mon, Jun 16, 2014 at 5:02 AM, Norbert Koch <viteno(a)xemacs.org> wrote:
Erm, has this changed in one of the last commits? I've done it
this way
since like forever ...
I don't know what changed, but package-net-packages.el has a bug that
is causing this, it seems to me. Function
package-net-packages-batch-convert-index-to-ini is using command line
arguments, but not marking them as used. Therefore, the last two
arguments on the command line, namely "/usr/users/nk/pkg-build/staging
standard" are processed by command-line once that function returns,
but since /usr/users/nk/pkg-build/staging is a directory, XEmacs
doesn't know what to do with it and complains.
I think this patch is needed:
diff -r 9954f5555c86 package-net-packages.el
--- a/package-net-packages.el Wed Jun 04 15:49:00 2014 -0600
+++ b/package-net-packages.el Tue Jun 17 10:46:32 2014 -0600
＠＠ -145,8 +145,8 ＠＠
"Convert the package index to ini file format."
(unless noninteractive
(error "`package-net-packages-batch-convert-index-to-ini' is to
be used only with -batch"))
- (let ((dir (car command-line-args-left))
- (category (cadr command-line-args-left))
+ (let ((dir (pop command-line-args-left))
+ (category (pop command-line-args-left))
(package-get-require-signed-base-updates nil))
(package-net-packages-convert-index-to-ini dir nil category)))
And maybe that let should be a let*? Regards,
--
Jerry James
http://www.jamezone.org/
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-beta
10:24 a.m.
* Jerry James <james(a)xemacs.org>:
Hello Jerry,
I don't know what changed, but package-net-packages.el has a bug
that
is causing this, it seems to me. Function
package-net-packages-batch-convert-index-to-ini is using command line
arguments, but not marking them as used. Therefore, the last two
arguments on the command line, namely "/usr/users/nk/pkg-build/staging
standard" are processed by command-line once that function returns,
but since /usr/users/nk/pkg-build/staging is a directory, XEmacs
doesn't know what to do with it and complains.
I think this patch is needed:
[...]
Thanks a lot! I've applied it by hand (too many mailers had their
fingers into this :-)) and now the make works.
And maybe that let should be a let*?
Sounds reasonable.
norbert.
--
_______________________________________________
XEmacs-Beta mailing list
XEmacs-Beta(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-beta
3810
days inactive
3812
days old
4 comments
3 participants
participants (3)
-
Jerry James -
Norbert Koch -
The XEmacs Package Smoketest