* ./make-dist for unicode branch @ 2006-01-17 15:43 Emfox Zhou 2006-01-17 19:56 ` Eli Zaretskii 0 siblings, 1 reply; 29+ messages in thread From: Emfox Zhou @ 2006-01-17 15:43 UTC (permalink / raw) when using ./make-dist --snapshot to generate a tarball for unicode branch, I got a .tar.gz without the 'admin' directory. Is there anything more I should do, or it's just because ./make-dist is deprecated? If in the latter case, what should I use? ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-17 15:43 ./make-dist for unicode branch Emfox Zhou @ 2006-01-17 19:56 ` Eli Zaretskii 2006-01-17 22:08 ` Emfox Zhou 0 siblings, 1 reply; 29+ messages in thread From: Eli Zaretskii @ 2006-01-17 19:56 UTC (permalink / raw) Cc: emacs-devel > From: Emfox Zhou <EmfoxZhou@gmail.com> > Date: Tue, 17 Jan 2006 23:43:41 +0800 > > when using ./make-dist --snapshot to generate a tarball for unicode branch, > I got a .tar.gz without the 'admin' directory. Is there anything more I > should do, or it's just because ./make-dist is deprecated? Neither: make-dist excludes the admin directory on purpose, since it is not for users and should not be in the distribution tarball. Why did you think you need it in a snapshot? ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-17 19:56 ` Eli Zaretskii @ 2006-01-17 22:08 ` Emfox Zhou 2006-01-18 0:41 ` Kenichi Handa 0 siblings, 1 reply; 29+ messages in thread From: Emfox Zhou @ 2006-01-17 22:08 UTC (permalink / raw) Eli Zaretskii <eliz@gnu.org> writes: > Neither: make-dist excludes the admin directory on purpose, since it > is not for users and should not be in the distribution tarball. > > Why did you think you need it in a snapshot? ar, but when I used a tarball from make-dist, it built error with some reference to admin/unidata, while it seems everything was ok by build emacs unicode branch directly in cvs checkouted directory. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-17 22:08 ` Emfox Zhou @ 2006-01-18 0:41 ` Kenichi Handa 2006-01-18 3:38 ` Emfox Zhou 0 siblings, 1 reply; 29+ messages in thread From: Kenichi Handa @ 2006-01-18 0:41 UTC (permalink / raw) Cc: emacs-devel In article <877j8yse2a.fsf@emfox.3322.org>, Emfox Zhou <EmfoxZhou@gmail.com> writes: > ar, but when I used a tarball from make-dist, it built > error with some reference to admin/unidata, while it seems > everything was ok by build emacs unicode branch directly > in cvs checkouted directory. Then, it means that I made some mistake in emacs-unicode-2 branch. Could you show me the exact error message? --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-18 0:41 ` Kenichi Handa @ 2006-01-18 3:38 ` Emfox Zhou 2006-01-19 11:35 ` Kenichi Handa 0 siblings, 1 reply; 29+ messages in thread From: Emfox Zhou @ 2006-01-18 3:38 UTC (permalink / raw) Kenichi Handa <handa@m17n.org> writes: > In article <877j8yse2a.fsf@emfox.3322.org>, Emfox Zhou <EmfoxZhou@gmail.com> writes: > >> ar, but when I used a tarball from make-dist, it built >> error with some reference to admin/unidata, while it seems >> everything was ok by build emacs unicode branch directly >> in cvs checkouted directory. > > Then, it means that I made some mistake in emacs-unicode-2 > branch. Could you show me the exact error message? I tried to make a debian package of emacs-unicode-2 branch, but it built error every time, so I started to build it from scratch, without debian/ files. (If that's right) I did a 'make bootstrap' after './configure', and here's the last several lines of the error msg: RUNEMACS="`/bin/pwd`/temacs"; cd /home/emfox/file/debian/packages/emacs/emacs-23.0.0/src/../admin/unidata; make -w RUNEMACS="${RUNEMACS}" DSTDIR=/home/emfox/file/debian/packages/emacs/emacs-23.0.0/src/../lisp/international /bin/sh: line 0: cd: /home/emfox/file/debian/packages/emacs/emacs-23.0.0/src/../admin/unidata: No such file or directory make[3]: Entering directory `/home/emfox/file/debian/packages/emacs/emacs-23.0.0/src' cd ../lwlib/; make -w CC='gcc' CFLAGS='-g -O2 -Wno-pointer-sign ' MAKE='make' "C_SWITCH_X_SITE=-I/usr/X11R6/include" "C_SWITCH_X_MACHINE=" "C_SWITCH_X_SYSTEM=" "C_SWITCH_SITE=" "C_SWITCH_MACHINE=" "C_SWITCH_SYSTEM=-D_BSD_SOURCE" make[4]: Entering directory `/home/emfox/file/debian/packages/emacs/emacs-23.0.0/lwlib' make[4]: Nothing to be done for `all'. make[4]: Leaving directory `/home/emfox/file/debian/packages/emacs/emacs-23.0.0/lwlib' cd ../lib-src/; make -w make-docfile make[4]: Entering directory `/home/emfox/file/debian/packages/emacs/emacs-23.0.0/lib-src' gcc -D_BSD_SOURCE -DHAVE_CONFIG_H -I. -I../src -I/home/emfox/file/debian/packages/emacs/emacs-23.0.0/lib-src -I/home/emfox/file/debian/packages/emacs/emacs-23.0.0/lib-src/../src -g -O2 -Wno-pointer-sign /home/emfox/file/debian/packages/emacs/emacs-23.0.0/lib-src/make-docfile.c -lc -o make-docfile make[4]: Leaving directory `/home/emfox/file/debian/packages/emacs/emacs-23.0.0/lib-src' make[3]: *** No rule to make target `/home/emfox/file/debian/packages/emacs/emacs-23.0.0/src/../lisp/abbrev.elc', needed by `../etc/DOC'. Stop. make[3]: Leaving directory `/home/emfox/file/debian/packages/emacs/emacs-23.0.0/src' make[2]: *** [/home/emfox/file/debian/packages/emacs/emacs-23.0.0/src/../lisp/international/charprop.el] Error 2make[2]: Leaving directory `/home/emfox/file/debian/packages/emacs/emacs-23.0.0/src' make[1]: *** [bootstrap-build] Error 2 make[1]: Leaving directory `/home/emfox/file/debian/packages/emacs/emacs-23.0.0' make: *** [bootstrap] Error 2 ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-18 3:38 ` Emfox Zhou @ 2006-01-19 11:35 ` Kenichi Handa 2006-01-19 19:05 ` Emfox Zhou 0 siblings, 1 reply; 29+ messages in thread From: Kenichi Handa @ 2006-01-19 11:35 UTC (permalink / raw) Cc: emacs-devel In article <873bjmryr5.fsf@emfox.3322.org>, Emfox Zhou <EmfoxZhou@gmail.com> writes: > I tried to make a debian package of emacs-unicode-2 branch, but it built error > every time, so I started to build it from scratch, without debian/ files. (If > that's right) I did a 'make bootstrap' after './configure', and here's the last > several lines of the error msg: > RUNEMACS="`/bin/pwd`/temacs"; cd /home/emfox/file/debian/packages/emacs/emacs-23.0.0/src/../admin/unidata; make -w RUNEMACS="${RUNEMACS}" DSTDIR=/home/emfox/file/debian/packages/emacs/emacs-23.0.0/src/../lisp/international > /bin/sh: line 0: cd: /home/emfox/file/debian/packages/emacs/emacs-23.0.0/src/../admin/unidata: No such file or directory I've installed several fixes to emacs-unicode-2. I made a tarball by ./make-dist --snapshot, and from that tarball, I could build emacs simply by "configure; make" and also by "configure; make bootstrap". --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-19 11:35 ` Kenichi Handa @ 2006-01-19 19:05 ` Emfox Zhou 2006-01-20 4:50 ` Kenichi Handa 0 siblings, 1 reply; 29+ messages in thread From: Emfox Zhou @ 2006-01-19 19:05 UTC (permalink / raw) Kenichi Handa <handa@m17n.org> writes: > I've installed several fixes to emacs-unicode-2. I made a > tarball by ./make-dist --snapshot, and from that tarball, I > could build emacs simply by "configure; make" and also by > "configure; make bootstrap". I'm sorry to tell that the same error as my last letter occurred. Here the command what EXACTLY I used: ======= cvs -z3 -d:pserver:anonymous@cvs.savannah.gnu.org:/cvsroot/emacs \ co -r emacs-unicode-2 -P emacs cd emacs ./make-dist --snapshot tar xzf emacs-23.0.0.tar.gz cd emacs-23.0.0/ ./configure make bootstrap ======= Please have a look at them, and correct if there's anything wrong, or any other more dependeny(such as different version of autotools) I should provide. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-19 19:05 ` Emfox Zhou @ 2006-01-20 4:50 ` Kenichi Handa 2006-01-20 4:58 ` Miles Bader ` (2 more replies) 0 siblings, 3 replies; 29+ messages in thread From: Kenichi Handa @ 2006-01-20 4:50 UTC (permalink / raw) Cc: emacs-devel In article <87vewgca2p.fsf@emfox.3322.org>, Emfox Zhou <EmfoxZhou@gmail.com> writes: > I'm sorry to tell that the same error as my last letter occurred. > Here the command what EXACTLY I used: > ======= > cvs -z3 -d:pserver:anonymous@cvs.savannah.gnu.org:/cvsroot/emacs \ > co -r emacs-unicode-2 -P emacs > cd emacs > ./make-dist --snapshot > tar xzf emacs-23.0.0.tar.gz > cd emacs-23.0.0/ > ./configure > make bootstrap > ======= Ah! I found what is wrong. The files lisp/international/charprop.el and lisp/international/uni-*.el are generated by "make bootstrap" if there exists "admin" subdir. But, as they don't exist in CVS, your tarball doesn't contain them (nor 'admin' subdir). I vaguely remember that I have committed all those files once. I don't know why they don't exist in CVS now. I may have deleted them by myself for some reason (I don't remember well). :-( Anyway, I've just committed them. Could you please try again? By the way, is it the right thing to run 'make-dist --snapshot' without building Emacs once in that directory? The resulting tarball may not contain *.elc. --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-20 4:50 ` Kenichi Handa @ 2006-01-20 4:58 ` Miles Bader 2006-01-20 6:30 ` Kenichi Handa 2006-01-20 6:13 ` Emfox Zhou 2006-01-20 10:11 ` Zhang Wei 2 siblings, 1 reply; 29+ messages in thread From: Miles Bader @ 2006-01-20 4:58 UTC (permalink / raw) Cc: emacs-devel, Emfox Zhou Kenichi Handa <handa@m17n.org> writes: > The files lisp/international/charprop.el and > lisp/international/uni-*.el are generated by "make bootstrap" if there > exists "admin" subdir. But, as they don't exist in CVS, your tarball > doesn't contain them (nor 'admin' subdir). > > I vaguely remember that I have committed all those files > once. I don't know why they don't exist in CVS now. If they're automatically generated, they shouldn't be in CVS... -Miles -- [|nurgle|] ddt- demonic? so quake will have an evil kinda setting? one that will make every christian in the world foamm at the mouth? [iddt] nurg, that's the goal ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-20 4:58 ` Miles Bader @ 2006-01-20 6:30 ` Kenichi Handa 2006-01-20 6:56 ` Miles Bader 2006-01-20 10:53 ` Eli Zaretskii 0 siblings, 2 replies; 29+ messages in thread From: Kenichi Handa @ 2006-01-20 6:30 UTC (permalink / raw) Cc: EmfoxZhou, emacs-devel In article <buor7735wc7.fsf@dhapc248.dev.necel.com>, Miles Bader <miles.bader@necel.com> writes: > Kenichi Handa <handa@m17n.org> writes: >> The files lisp/international/charprop.el and >> lisp/international/uni-*.el are generated by "make bootstrap" if there >> exists "admin" subdir. But, as they don't exist in CVS, your tarball >> doesn't contain them (nor 'admin' subdir). >> >> I vaguely remember that I have committed all those files >> once. I don't know why they don't exist in CVS now. > If they're automatically generated, they shouldn't be in CVS... I basically agree with that. But, then, how can we make them included in a tarball generated by Emfox's way? Shuold we signal an error in make-dist when the directory is not yet readly for creating a tar ball? But it's not trivial to check if the directory is ready or not. --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-20 6:30 ` Kenichi Handa @ 2006-01-20 6:56 ` Miles Bader 2006-01-20 10:53 ` Eli Zaretskii 1 sibling, 0 replies; 29+ messages in thread From: Miles Bader @ 2006-01-20 6:56 UTC (permalink / raw) Cc: EmfoxZhou, emacs-devel Kenichi Handa <handa@m17n.org> writes: >> If they're automatically generated, they shouldn't be in CVS... > > I basically agree with that. But, then, how can we make > them included in a tarball generated by Emfox's way? I admit, I have no idea what the intended purpose of "--snapshot" is, but for the normal case (no --snapshot), make-dist contains logic to build any files that need to be generated (.elc files, info files, etc), by calling "make" or invoking emacs or whatever. It seems that this data should be handled in the same way. Since --snapshot results in "--no-update --no-check" it would seem to be imply that it's up to the user to make sure their tree is in a sane state. -Miles -- We are all lying in the gutter, but some of us are looking at the stars. -Oscar Wilde ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-20 6:30 ` Kenichi Handa 2006-01-20 6:56 ` Miles Bader @ 2006-01-20 10:53 ` Eli Zaretskii 2006-01-26 1:09 ` Kenichi Handa 1 sibling, 1 reply; 29+ messages in thread From: Eli Zaretskii @ 2006-01-20 10:53 UTC (permalink / raw) Cc: EmfoxZhou, emacs-devel, miles > From: Kenichi Handa <handa@m17n.org> > Date: Fri, 20 Jan 2006 15:30:33 +0900 > Cc: EmfoxZhou@gmail.com, emacs-devel@gnu.org > > In article <buor7735wc7.fsf@dhapc248.dev.necel.com>, Miles Bader <miles.bader@necel.com> writes: > > > Kenichi Handa <handa@m17n.org> writes: > >> The files lisp/international/charprop.el and > >> lisp/international/uni-*.el are generated by "make bootstrap" if there > >> exists "admin" subdir. But, as they don't exist in CVS, your tarball > >> doesn't contain them (nor 'admin' subdir). > >> > >> I vaguely remember that I have committed all those files > >> once. I don't know why they don't exist in CVS now. > > > If they're automatically generated, they shouldn't be in CVS... > > I basically agree with that. But, then, how can we make > them included in a tarball generated by Emfox's way? What is the reason that those files are generated only if there's the admin/ directory? > Shuold we signal an error in make-dist when the directory is > not yet readly for creating a tar ball? But it's not > trivial to check if the directory is ready or not. I think a snapshot should be exactly like a release tarball, i.e. it should include all the *.elc files etc. So one needs to build Emacs before creating a snapshot. In other words, a snapshot is NOT simply a checked-out CVS tree, it's a tarball like any release, except that it was made from a non-released code base. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-20 10:53 ` Eli Zaretskii @ 2006-01-26 1:09 ` Kenichi Handa 2006-01-26 17:46 ` Richard M. Stallman 0 siblings, 1 reply; 29+ messages in thread From: Kenichi Handa @ 2006-01-26 1:09 UTC (permalink / raw) Cc: EmfoxZhou, emacs-devel, miles In article <uslrjqifz.fsf@gnu.org>, Eli Zaretskii <eliz@gnu.org> writes: > I think a snapshot should be exactly like a release tarball, i.e. it > should include all the *.elc files etc. So one needs to build Emacs > before creating a snapshot. > In other words, a snapshot is NOT simply a checked-out CVS tree, it's > a tarball like any release, except that it was made from a > non-released code base. If that is the rule, I don't mind removing uni-*.el from the CVS tree again. Richard, shall I? --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-26 1:09 ` Kenichi Handa @ 2006-01-26 17:46 ` Richard M. Stallman 2006-01-26 21:21 ` Eli Zaretskii 0 siblings, 1 reply; 29+ messages in thread From: Richard M. Stallman @ 2006-01-26 17:46 UTC (permalink / raw) Cc: miles, eliz, emacs-devel, EmfoxZhou If that is the rule, I don't mind removing uni-*.el from the CVS tree again. Richard, shall I? I was not following this discussion and I don't know what the issue is. Eli, could you summarize it? ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-26 17:46 ` Richard M. Stallman @ 2006-01-26 21:21 ` Eli Zaretskii 2006-01-27 0:49 ` Kenichi Handa 2006-01-27 22:32 ` Richard M. Stallman 0 siblings, 2 replies; 29+ messages in thread From: Eli Zaretskii @ 2006-01-26 21:21 UTC (permalink / raw) Cc: EmfoxZhou, emacs-devel, handa > From: "Richard M. Stallman" <rms@gnu.org> > CC: eliz@gnu.org, EmfoxZhou@gmail.com, emacs-devel@gnu.org, > miles@gnu.org > Date: Thu, 26 Jan 2006 12:46:43 -0500 > > If that is the rule, I don't mind removing uni-*.el from the > CVS tree again. Richard, shall I? > > I was not following this discussion and I don't know what the issue is. > Eli, could you summarize it? First, this is about the unicode-2 branch, not about CVS HEAD. The problem is that a tarball created with "make-dist --snapshot" out of a freshly checked-out CVS unicode-2 branch tree would not build because the files lisp/international/charprop.el and lisp/international/uni-*.el are generated by "make bootstrap" only if there's the `admin' subdirectory. (I asked why it looks for that directory, but didn't see any answers.) Handa-san tried to fix this by checking the missing files into CVS, but Miles objected, saying that generated files should not be there. I think this is a non-issue, because make-dist is not supposed to produce a workable tarball unless you bootstrap first. See admin/make-tarball.txt, it explicitly says to bootstrap. As I see this, a snapshot should be exactly like a release tarball, i.e. it should include all the *.elc files etc. So one needs to build Emacs before creating a snapshot. So I don't think these files should be in CVS, and I don't think there's any problem in make-dist. Rather, before creating a snapshot, people should say "make bootstrap" in their sandbox. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-26 21:21 ` Eli Zaretskii @ 2006-01-27 0:49 ` Kenichi Handa 2006-01-27 7:23 ` Eli Zaretskii 2006-01-27 22:32 ` Richard M. Stallman 1 sibling, 1 reply; 29+ messages in thread From: Kenichi Handa @ 2006-01-27 0:49 UTC (permalink / raw) Cc: emacs-devel, rms, EmfoxZhou In article <uk6cmeld0.fsf@gnu.org>, Eli Zaretskii <eliz@gnu.org> writes: > The problem is that a tarball created with "make-dist --snapshot" out > of a freshly checked-out CVS unicode-2 branch tree would not build > because the files lisp/international/charprop.el and > lisp/international/uni-*.el are generated by "make bootstrap" only if > there's the `admin' subdirectory. (I asked why it looks for that > directory, but didn't see any answers.) Ah, sorry for forgetting to answer it. The reason is that those files are generated from admin/unidata/UnicodeData.txt. This file is about 1M byte, so I thought it was too big to be included in a tarball. The total of generated uni-*.el is about 200K. > Handa-san tried to fix this by checking the missing files into CVS, > but Miles objected, saying that generated files should not be there. > I think this is a non-issue, because make-dist is not supposed to > produce a workable tarball unless you bootstrap first. See > admin/make-tarball.txt, it explicitly says to bootstrap. As I see > this, a snapshot should be exactly like a release tarball, i.e. it > should include all the *.elc files etc. So one needs to build Emacs > before creating a snapshot. > So I don't think these files should be in CVS, and I don't think > there's any problem in make-dist. Rather, before creating a snapshot, > people should say "make bootstrap" in their sandbox. I didn't know about admin/make-tarball.txt. Now I agree that we don't have to put uni-*.el in CVS tree. Perhaps, we should add a comment about the requirement of bootstrap at somewhere in make-dist. --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-27 0:49 ` Kenichi Handa @ 2006-01-27 7:23 ` Eli Zaretskii 2006-01-27 7:47 ` Kenichi Handa 0 siblings, 1 reply; 29+ messages in thread From: Eli Zaretskii @ 2006-01-27 7:23 UTC (permalink / raw) Cc: emacs-devel, rms, EmfoxZhou > From: Kenichi Handa <handa@m17n.org> > CC: rms@gnu.org, EmfoxZhou@gmail.com, emacs-devel@gnu.org > Date: Fri, 27 Jan 2006 09:49:49 +0900 > > > The problem is that a tarball created with "make-dist --snapshot" out > > of a freshly checked-out CVS unicode-2 branch tree would not build > > because the files lisp/international/charprop.el and > > lisp/international/uni-*.el are generated by "make bootstrap" only if > > there's the `admin' subdirectory. (I asked why it looks for that > > directory, but didn't see any answers.) > > Ah, sorry for forgetting to answer it. The reason is that > those files are generated from > admin/unidata/UnicodeData.txt. This file is about 1M byte, > so I thought it was too big to be included in a tarball. > The total of generated uni-*.el is about 200K. It's too early to discuss the packaging of Emacs 23, but I'd say UnicodeData.txt _should_ be part of the tarball, because otherwise users will not be able to regenerate those *.el files if they want to change some of the characters' attributes. 800KB more is not going to change anything in what is already a 80MB to 130MB source tree. > I didn't know about admin/make-tarball.txt. Now I agree > that we don't have to put uni-*.el in CVS tree. Perhaps, we > should add a comment about the requirement of bootstrap at > somewhere in make-dist. Even better, perhaps make-dist should find some way to check whether a bootstrap was done, and refuse to work if it wasn't. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-27 7:23 ` Eli Zaretskii @ 2006-01-27 7:47 ` Kenichi Handa 2006-01-28 4:50 ` Richard M. Stallman 0 siblings, 1 reply; 29+ messages in thread From: Kenichi Handa @ 2006-01-27 7:47 UTC (permalink / raw) Cc: emacs-devel, rms, EmfoxZhou In article <uhd7qdth2.fsf@gnu.org>, Eli Zaretskii <eliz@gnu.org> writes: > It's too early to discuss the packaging of Emacs 23, but I'd say > UnicodeData.txt _should_ be part of the tarball, because otherwise > users will not be able to regenerate those *.el files if they want > to change some of the characters' attributes. For that, they dont' have to regenerate uni-*.el files. They can change each attribute by `put-char-code-property'. >> I didn't know about admin/make-tarball.txt. Now I agree >> that we don't have to put uni-*.el in CVS tree. Perhaps, we >> should add a comment about the requirement of bootstrap at >> somewhere in make-dist. > Even better, perhaps make-dist should find some way to check whether a > bootstrap was done, and refuse to work if it wasn't. Yes, but, as I wrote before, I think it's not a trivial work. --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-27 7:47 ` Kenichi Handa @ 2006-01-28 4:50 ` Richard M. Stallman 2006-01-29 12:11 ` Kenichi Handa 0 siblings, 1 reply; 29+ messages in thread From: Richard M. Stallman @ 2006-01-28 4:50 UTC (permalink / raw) Cc: eliz, EmfoxZhou, emacs-devel > Even better, perhaps make-dist should find some way to check whether a > bootstrap was done, and refuse to work if it wasn't. Yes, but, as I wrote before, I think it's not a trivial work. I think it is trivial. Just check whether simple.elc exists. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-28 4:50 ` Richard M. Stallman @ 2006-01-29 12:11 ` Kenichi Handa 2006-01-30 0:57 ` Richard M. Stallman 0 siblings, 1 reply; 29+ messages in thread From: Kenichi Handa @ 2006-01-29 12:11 UTC (permalink / raw) Cc: eliz, EmfoxZhou, emacs-devel In article <E1F2i39-0003bo-FT@fencepost.gnu.org>, "Richard M. Stallman" <rms@gnu.org> writes: >> Even better, perhaps make-dist should find some way to check whether a >> bootstrap was done, and refuse to work if it wasn't. > Yes, but, as I wrote before, I think it's not a trivial work. > I think it is trivial. Just check whether simple.elc exists. It doesn't assure that the bootstrap was really successful. --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-29 12:11 ` Kenichi Handa @ 2006-01-30 0:57 ` Richard M. Stallman 2006-01-30 12:32 ` Kenichi Handa 0 siblings, 1 reply; 29+ messages in thread From: Richard M. Stallman @ 2006-01-30 0:57 UTC (permalink / raw) Cc: eliz, EmfoxZhou, emacs-devel > I think it is trivial. Just check whether simple.elc exists. It doesn't assure that the bootstrap was really successful. Does that really matter here? The point is to detect the case where no bootstrap was done. It would be nice to detect the case of a failed bootstrap also, but that is a different matter. It is ok to detect just the former. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-30 0:57 ` Richard M. Stallman @ 2006-01-30 12:32 ` Kenichi Handa 2006-01-30 19:23 ` Eli Zaretskii ` (2 more replies) 0 siblings, 3 replies; 29+ messages in thread From: Kenichi Handa @ 2006-01-30 12:32 UTC (permalink / raw) Cc: eliz, emacs-devel, EmfoxZhou In article <E1F3NLx-0000H7-RS@fencepost.gnu.org>, "Richard M. Stallman" <rms@gnu.org> writes: >> I think it is trivial. Just check whether simple.elc exists. > It doesn't assure that the bootstrap was really successful. > Does that really matter here? > The point is to detect the case where no bootstrap was done. > It would be nice to detect the case of a failed bootstrap > also, but that is a different matter. It is ok to detect > just the former. I thought that the point was to prevent making an incomplete tarball. And, for that, it is necessary to detect the case where bootstrap failed. The merit of just checking if bootstrap was done or not regardless of the result is small, isn't it? --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-30 12:32 ` Kenichi Handa @ 2006-01-30 19:23 ` Eli Zaretskii 2006-01-31 1:08 ` Kenichi Handa 2006-01-31 1:51 ` Miles Bader 2006-01-31 18:04 ` Richard M. Stallman 2 siblings, 1 reply; 29+ messages in thread From: Eli Zaretskii @ 2006-01-30 19:23 UTC (permalink / raw) Cc: emacs-devel, rms, EmfoxZhou > From: Kenichi Handa <handa@m17n.org> > CC: eliz@gnu.org, EmfoxZhou@gmail.com, emacs-devel@gnu.org > Date: Mon, 30 Jan 2006 21:32:37 +0900 > > In article <E1F3NLx-0000H7-RS@fencepost.gnu.org>, "Richard M. Stallman" <rms@gnu.org> writes: > > >> I think it is trivial. Just check whether simple.elc exists. > > It doesn't assure that the bootstrap was really successful. > > > Does that really matter here? > > > The point is to detect the case where no bootstrap was done. > > It would be nice to detect the case of a failed bootstrap > > also, but that is a different matter. It is ok to detect > > just the former. > > I thought that the point was to prevent making an incomplete > tarball. And, for that, it is necessary to detect the case > where bootstrap failed. The merit of just checking if > bootstrap was done or not regardless of the result is small, > isn't it? How about if we check that all the generated source files that are not in the repository were regenerated? (Excluding the *.elc, *.o, and other compiled files, of course.) There shouldn't be too many, I think. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-30 19:23 ` Eli Zaretskii @ 2006-01-31 1:08 ` Kenichi Handa 0 siblings, 0 replies; 29+ messages in thread From: Kenichi Handa @ 2006-01-31 1:08 UTC (permalink / raw) Cc: EmfoxZhou, rms, emacs-devel In article <u3bj5ldti.fsf@gnu.org>, Eli Zaretskii <eliz@gnu.org> writes: > How about if we check that all the generated source files that are not > in the repository were regenerated? (Excluding the *.elc, *.o, and > other compiled files, of course.) There shouldn't be too many, I > think. Perhaps. But, I suggest to postpone the further discussion on this matter because the problem is in emacs-unicode-2 branch only for the moment. How about putting these entries in TODO? * Make the role of "./make-dist --snapshot" clear and, if necessary, fix make-dist to do that correctly. * Decide how to handle admin/unidata/UnicodeData.txt and the generated files lisp/international/uni-*.el. And, as this is administrative matter, that kind of TODO file should be under "admin" subdir. --- Kenichi Handa handa@m17n.org ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-30 12:32 ` Kenichi Handa 2006-01-30 19:23 ` Eli Zaretskii @ 2006-01-31 1:51 ` Miles Bader 2006-01-31 18:04 ` Richard M. Stallman 2 siblings, 0 replies; 29+ messages in thread From: Miles Bader @ 2006-01-31 1:51 UTC (permalink / raw) Cc: eliz, EmfoxZhou, rms, emacs-devel 2006/1/30, Kenichi Handa <handa@m17n.org>: > I thought that the point was to prevent making an incomplete > tarball. And, for that, it is necessary to detect the case > where bootstrap failed. The merit of just checking if > bootstrap was done or not regardless of the result is small, > isn't it? Why do you think that? I'd think by far the most common case would be people who checked out but didn't build (i.e. the easy-to-detect case). Emacs does not attempt to do a rigorous up-to-date check anywhere else, why should it do that here? That's especially true given the low frequency of changes in unicode data files -- if the generated files exist at all, they're probably good enough to use for a snapshot. -miles -- Do not taunt Happy Fun Ball. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-30 12:32 ` Kenichi Handa 2006-01-30 19:23 ` Eli Zaretskii 2006-01-31 1:51 ` Miles Bader @ 2006-01-31 18:04 ` Richard M. Stallman 2 siblings, 0 replies; 29+ messages in thread From: Richard M. Stallman @ 2006-01-31 18:04 UTC (permalink / raw) Cc: eliz, emacs-devel, EmfoxZhou I thought that the point was to prevent making an incomplete tarball. The aim of this suggestion is to catch a simple mistake of running make-dist without bootstrapping first. To check for the other case of a bootstrap that failed is a different issue. If someone wants to do it, I don't object. If people don't want to bother, that's fine too. In any case, my suggestion is to test that simple.elc exists and give an error if it does not. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-26 21:21 ` Eli Zaretskii 2006-01-27 0:49 ` Kenichi Handa @ 2006-01-27 22:32 ` Richard M. Stallman 1 sibling, 0 replies; 29+ messages in thread From: Richard M. Stallman @ 2006-01-27 22:32 UTC (permalink / raw) Cc: EmfoxZhou, emacs-devel, handa because the files lisp/international/charprop.el and lisp/international/uni-*.el are generated by "make bootstrap" only if there's the `admin' subdirectory. (I asked why it looks for that directory, but didn't see any answers.) That seems really ugly. I think we should track down the reason for that conditional, and either document it or remove it (and solve whatever problem in some other way). Can you do that? Handa-san tried to fix this by checking the missing files into CVS, but Miles objected, saying that generated files should not be there. I agree these files should not be in CVS. So I don't think these files should be in CVS, and I don't think there's any problem in make-dist. Rather, before creating a snapshot, people should say "make bootstrap" in their sandbox. If this is so, make-dist should check that a bootstrap has been done, and fail with a clear error message otherwise. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-20 4:50 ` Kenichi Handa 2006-01-20 4:58 ` Miles Bader @ 2006-01-20 6:13 ` Emfox Zhou 2006-01-20 10:11 ` Zhang Wei 2 siblings, 0 replies; 29+ messages in thread From: Emfox Zhou @ 2006-01-20 6:13 UTC (permalink / raw) Kenichi Handa <handa@m17n.org> writes: > > Ah! I found what is wrong. The files > lisp/international/charprop.el and > lisp/international/uni-*.el are generated by "make > bootstrap" if there exists "admin" subdir. But, as they > don't exist in CVS, your tarball doesn't contain them (nor > 'admin' subdir). > > I vaguely remember that I have committed all those files > once. I don't know why they don't exist in CVS now. I may > have deleted them by myself for some reason (I don't > remember well). :-( Anyway, I've just committed them. Could > you please try again? > Yes, I works perfectly now. > By the way, is it the right thing to run 'make-dist > --snapshot' without building Emacs once in that directory? > The resulting tarball may not contain *.elc. > It's just a 'snapshot' of cvs version, nothing serious. ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: ./make-dist for unicode branch 2006-01-20 4:50 ` Kenichi Handa 2006-01-20 4:58 ` Miles Bader 2006-01-20 6:13 ` Emfox Zhou @ 2006-01-20 10:11 ` Zhang Wei 2 siblings, 0 replies; 29+ messages in thread From: Zhang Wei @ 2006-01-20 10:11 UTC (permalink / raw) Cc: emacs-devel, Emfox Zhou Kenichi Handa <handa@m17n.org> writes: [...] > I vaguely remember that I have committed all those files > once. I don't know why they don't exist in CVS now. I may > have deleted them by myself for some reason (I don't > remember well). :-( Anyway, I've just committed them. Could > you please try again? The committed uni-* files make cvs very noisy while I do "cvs update". [...] ^ permalink raw reply [flat|nested] 29+ messages in thread
end of thread, other threads:[~2006-01-31 18:04 UTC | newest] Thread overview: 29+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2006-01-17 15:43 ./make-dist for unicode branch Emfox Zhou 2006-01-17 19:56 ` Eli Zaretskii 2006-01-17 22:08 ` Emfox Zhou 2006-01-18 0:41 ` Kenichi Handa 2006-01-18 3:38 ` Emfox Zhou 2006-01-19 11:35 ` Kenichi Handa 2006-01-19 19:05 ` Emfox Zhou 2006-01-20 4:50 ` Kenichi Handa 2006-01-20 4:58 ` Miles Bader 2006-01-20 6:30 ` Kenichi Handa 2006-01-20 6:56 ` Miles Bader 2006-01-20 10:53 ` Eli Zaretskii 2006-01-26 1:09 ` Kenichi Handa 2006-01-26 17:46 ` Richard M. Stallman 2006-01-26 21:21 ` Eli Zaretskii 2006-01-27 0:49 ` Kenichi Handa 2006-01-27 7:23 ` Eli Zaretskii 2006-01-27 7:47 ` Kenichi Handa 2006-01-28 4:50 ` Richard M. Stallman 2006-01-29 12:11 ` Kenichi Handa 2006-01-30 0:57 ` Richard M. Stallman 2006-01-30 12:32 ` Kenichi Handa 2006-01-30 19:23 ` Eli Zaretskii 2006-01-31 1:08 ` Kenichi Handa 2006-01-31 1:51 ` Miles Bader 2006-01-31 18:04 ` Richard M. Stallman 2006-01-27 22:32 ` Richard M. Stallman 2006-01-20 6:13 ` Emfox Zhou 2006-01-20 10:11 ` Zhang Wei
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/emacs.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).