emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* Please test the new Makefile
@ 2011-12-21 18:47 Bastien
  2011-12-22  1:35 ` Takaaki ISHIKAWA
  2011-12-22 10:25 ` Achim Gratz
  0 siblings, 2 replies; 12+ messages in thread
From: Bastien @ 2011-12-21 18:47 UTC (permalink / raw)
  To: emacs-orgmode

Dear all,

I've now bundled Achim's new Makefile in these archives:

  http://orgmode.org/Org-7.8.03-TestMakefile.tar.gz
  http://orgmode.org/Org-7.8.03-TestMakefile.zip

Can some of you test the make procedures from there?

Things are fine here, you should be safe.  The purpose of 
the test is to make sure `make && make install' do the right
thing in various setups.  

Thanks for your help,

Ps: for those who want to test this from Achim's branch, 
here is the recipe, assuming you are in your org-mode dir:

git remote add -t Makefile remote-tableheadings git://repo.or.cz/org-mode/org-tableheadings.git
git fetch remote-tableheadings Makefile:local-Makefile
git checkout local-Makefile

-- 
 Bastien

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-21 18:47 Please test the new Makefile Bastien
@ 2011-12-22  1:35 ` Takaaki ISHIKAWA
  2011-12-22 10:05   ` Achim Gratz
  2011-12-22 10:25 ` Achim Gratz
  1 sibling, 1 reply; 12+ messages in thread
From: Takaaki ISHIKAWA @ 2011-12-22  1:35 UTC (permalink / raw)
  To: Bastien; +Cc: emacs-orgmode

Hi Bastien,

I have tried to install the new package in my clean VM of Suse 12.1.
I'd like to report my install experience.

> http://orgmode.org/Org-7.8.03-TestMakefile.zip

1. When I just type "make", I find a fatal error message:
  Not a git repository (or any parent up to mount parent )
  Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

2. "make all" works well with the same message above.
  "make all" will be the default command instead of "make"?

3. Packages are installed into "/usr/share/emacs/site-lisp".
  This is a known change in ML.

4. "M-x org-version" shows:
  ( @ /usr/share/emacs/site-lisp/org/org-install.el)

Best regards,
Takaaki Ishikawa

----
GNU Emacs 23.3.1 (x86_64-suse-linux-gnu, GTK+ Version 2.24.7) of 2011-10-30 on build17

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-22  1:35 ` Takaaki ISHIKAWA
@ 2011-12-22 10:05   ` Achim Gratz
  2011-12-22 15:06     ` Takaaki ISHIKAWA
  2012-01-02 21:50     ` Achim Gratz
  0 siblings, 2 replies; 12+ messages in thread
From: Achim Gratz @ 2011-12-22 10:05 UTC (permalink / raw)
  To: emacs-orgmode

Am 22.12.2011 02:35, schrieb Takaaki ISHIKAWA:
> 1. When I just type "make", I find a fatal error message:
>    Not a git repository (or any parent up to mount parent )
>    Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

Just "make" doesn't work anymore (it displays a usage section to conform 
to GNU convention), you'll need "make all".  This now implies a "make 
clean", so you'll normally don't need this as an extra step anymore.

> 2. "make all" works well with the same message above.
>    "make all" will be the default command instead of "make"?

Yes.

> 3. Packages are installed into "/usr/share/emacs/site-lisp".
>    This is a known change in ML.

If you want it installed someplace else, create a file "local.mk" and 
override the variables in "default.mk".  So under Linux (assuming you're 
not logged in as root), you would do

make all && sudo make install

> 4. "M-x org-version" shows:
>    ( @ /usr/share/emacs/site-lisp/org/org-install.el)

I have not yet safeguarded against the situation where make is 
available, but not git.  You can also override this temporarily in 
targets.mk (GITVERSION, ORGVERSION and GITSTATUS).  I will re-arrange 
this to either test for a Git directory or make it overridable via 
"local.mk" (or both).

Thank you for testing!

-- 
Achim.

(on the road :-)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-21 18:47 Please test the new Makefile Bastien
  2011-12-22  1:35 ` Takaaki ISHIKAWA
@ 2011-12-22 10:25 ` Achim Gratz
  1 sibling, 0 replies; 12+ messages in thread
From: Achim Gratz @ 2011-12-22 10:25 UTC (permalink / raw)
  To: emacs-orgmode

Am 21.12.2011 19:47, schrieb Bastien:

> Things are fine here, you should be safe.  The purpose of
> the test is to make sure `make&&  make install' do the right
> thing in various setups.

Actually, that should be

make all && make install

or (under Unixoid system and running as a normal user):

make all && sudo make install

Before doing a "make install", please check "default.mk" for where it 
tries to put things.  You can override this by creating a file 
"local.mk" which just has definitions for those variables that you want 
to change (most of the time, just "prefix").

The installation is put into its own sub-directory "org" by default, if 
you've had a previous installation "flat" in site-lisp, you either need 
to remove those files or also install "flat" again - otherwise you will 
very likely pick up stale files from the previous installation.

If you are brave, there is a non-documented target "clean-install" that 
will remove those files, but you should make doubly sure that lispdir 
and infodir point at the right place (run "make -n clean-install" and 
check which files make would remove if you ran it without "-n").

> git remote add -t Makefile remote-tableheadings git://repo.or.cz/org-mode/org-tableheadings.git
> git fetch remote-tableheadings Makefile:local-Makefile
> git checkout local-Makefile

It is currently easier to test this from Git, since the makefile 
actually assumes a Git directory to be able to get at the version 
strings.  If you get bitten by this, please change the following lines 
in default.mk:

GITVERSION	= $(shell git describe --abbrev=6 HEAD)
ORGVERSION	= $(subst release_,,$(shell git describe --abbrev=0 HEAD))
GITSTATUS	= $(shell git status -uno --porcelain)

to

GITVERSION	= "release_7.8.03-Makefile
ORGVERSION	= "7.8.03"
GITSTATUS	= ""

I'm currently not able to rebase the branch on repo.or.cz, but you can 
do this yourself if you have your own local branch and also mirror the 
normal orgmode.git:

git remote update
git rebase local-Makefile origin/master

(Hope I got this right, I'm currently not able to test it myself :-)


-- 
Achim.

(on the road :-)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-22 10:05   ` Achim Gratz
@ 2011-12-22 15:06     ` Takaaki ISHIKAWA
  2011-12-23 16:10       ` Bastien
  2011-12-25 10:27       ` ASSI
  2012-01-02 21:50     ` Achim Gratz
  1 sibling, 2 replies; 12+ messages in thread
From: Takaaki ISHIKAWA @ 2011-12-22 15:06 UTC (permalink / raw)
  To: Achim Gratz; +Cc: emacs-orgmode

Dear Achim,

Hi. Thank you for your comment and great work on refactoring Makefile.

> Just "make" doesn't work anymore (it displays a usage section to conform to GNU convention), you'll need "make all".  This now implies a "make clean", so you'll normally don't need this as an extra step anymore.

I see why you use "make all".

Could you tell me the current trend of Makefile?
I think "make" and "make all" have the same effect usually.
In the new Makefile of org-mode, however, "make" is used 
for displaying options of make command.
Is this familiar to all?

For example, GNU Emacs doesn't require "make all", just "make".

Best regards,
Takaaki Ishikawa

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-22 15:06     ` Takaaki ISHIKAWA
@ 2011-12-23 16:10       ` Bastien
  2011-12-25 10:27       ` ASSI
  1 sibling, 0 replies; 12+ messages in thread
From: Bastien @ 2011-12-23 16:10 UTC (permalink / raw)
  To: Takaaki ISHIKAWA; +Cc: Achim Gratz, emacs-orgmode

Hi Achim and Takaaki,

Takaaki ISHIKAWA <takaxp@ieee.org> writes:

> I think "make" and "make all" have the same effect usually.
> In the new Makefile of org-mode, however, "make" is used 
> for displaying options of make command.
> Is this familiar to all?
>
> For example, GNU Emacs doesn't require "make all", just "make".

If we can stick to "make" let's stick to it.

If it breaks GNU coding conventions, let's use "make all" instead, 
and send a bug report to emacs-devel...

Thanks,

-- 
 Bastien

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-22 15:06     ` Takaaki ISHIKAWA
  2011-12-23 16:10       ` Bastien
@ 2011-12-25 10:27       ` ASSI
  2011-12-26  4:58         ` Takaaki ISHIKAWA
  1 sibling, 1 reply; 12+ messages in thread
From: ASSI @ 2011-12-25 10:27 UTC (permalink / raw)
  To: emacs-orgmode

[sorry if you get this twice
  - GMane seems to have trouble sending lately]

Am 22.12.2011 16:06, schrieb Takaaki ISHIKAWA:
> I think "make" and "make all" have the same effect usually.

Often they do, but GNU says that a simple "make" should behave like 
"make help".  I think this is appropriate for org mode since there is no 
INSTALL file that explains what to do and in most cases one would have 
to put some things in local.mk before running "make install".

> In the new Makefile of org-mode, however, "make" is used
> for displaying options of make command.
> Is this familiar to all?

I opted for this behaviour to align more closely to GNU convention and 
to avoid any surprising behaviour, but it is easy to make the default 
target "all" instead of "help".

> For example, GNU Emacs doesn't require "make all", just "make".

That's not exactly comparable since the Makefile for Emacs is the result 
of ./configure, so for Emacs make is just another tool used in a larger 
build system.


-- 
Achim.

(on the road :-)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-25 10:27       ` ASSI
@ 2011-12-26  4:58         ` Takaaki ISHIKAWA
  2011-12-27 10:33           ` Achim Gratz
  0 siblings, 1 reply; 12+ messages in thread
From: Takaaki ISHIKAWA @ 2011-12-26  4:58 UTC (permalink / raw)
  To: ASSI; +Cc: emacs-orgmode

Hi. Achim,

Thank you for your kind explanations.
I learned a lot from you :)

> I opted for this behaviour to align more closely to GNU convention and to avoid any surprising behaviour, but it is easy to make the default target "all" instead of "help".

I think many Org users are familiar to the previous
installation procedure by "make" not "make all",
and some users wrote it in their blog, Org manual [*1] too.
Don't you think this change has a wide impact for them?
But it's just a concern to me :)

[*1] http://orgmode.org/manual/Installation.html

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-26  4:58         ` Takaaki ISHIKAWA
@ 2011-12-27 10:33           ` Achim Gratz
  0 siblings, 0 replies; 12+ messages in thread
From: Achim Gratz @ 2011-12-27 10:33 UTC (permalink / raw)
  To: emacs-orgmode

Am 26.12.2011 05:58, schrieb Takaaki ISHIKAWA:
> I think many Org users are familiar to the previous installation
> procedure by "make" not "make all", and some users wrote it in their
> blog, Org manual [*1] too.

Yes, the manual would need to be changed in any case.  The semantics of 
some  targets have changed as well, even when they are having the same name.

> Don't you think this change has a wide impact for them? But it's just
> a concern to me :)

Again, if the consensus is that the default target should be "all", this
is just a single line to add at the top of the main Makefile:

.DEFAULT:	all

There are a few other changes I will have to implement, buit it will be
in the new year.

-- 
Achim.

(on the road :-)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2011-12-22 10:05   ` Achim Gratz
  2011-12-22 15:06     ` Takaaki ISHIKAWA
@ 2012-01-02 21:50     ` Achim Gratz
  2012-01-03 18:09       ` Achim Gratz
  1 sibling, 1 reply; 12+ messages in thread
From: Achim Gratz @ 2012-01-02 21:50 UTC (permalink / raw)
  To: emacs-orgmode


I've implemented a few improvements in my fork that should take care of
most issues raised in recent discussions:

- default target is now "all" instead of "help"

- make does not complain when not inside a Git tree

- all installations prefixed with $(DESTDIR) to allow for staged
  installation

- introduced $(datadir) as replacement for $(etcdir) and moved into
  default.mk; $(datadir) is recorded as a defconst into org-install.el

- more robust shell code

- removed maint.mk and maint-targets.mk, these will have to be rolled
  into local.mk (or included from there) on the server


I'm uncertain about the integration of the ODT exporter (although it
works on my machine): it seems that the schema files should reside in
etc/schema (and ETCDIR should include schema on install)?  Why are the
schema files only in contrib?  Also there is some compile-time
evaluation of org-odt-data-dir that seems unnecessary to me (I don't set
this variable at compile-time, anyhow)?



Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

DIY Stuff:
http://Synth.Stromeko.net/DIY.html

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2012-01-02 21:50     ` Achim Gratz
@ 2012-01-03 18:09       ` Achim Gratz
  2012-01-04 17:46         ` Achim Gratz
  0 siblings, 1 reply; 12+ messages in thread
From: Achim Gratz @ 2012-01-03 18:09 UTC (permalink / raw)
  To: emacs-orgmode


Rebased to 7.8.03.

Achim Gratz <Stromeko@nexgo.de> writes:
> I'm uncertain about the integration of the ODT exporter (although it
> works on my machine): it seems that the schema files should reside in
> etc/schema (and ETCDIR should include schema on install)? 

I went ahead and moved the schema dir from contrib to etc and adapted
the Makefile to install it.


Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

SD adaptation for Waldorf rackAttack V1.04R1:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: Please test the new Makefile
  2012-01-03 18:09       ` Achim Gratz
@ 2012-01-04 17:46         ` Achim Gratz
  0 siblings, 0 replies; 12+ messages in thread
From: Achim Gratz @ 2012-01-04 17:46 UTC (permalink / raw)
  To: emacs-orgmode

Achim Gratz <Stromeko@nexgo.de> writes:
> I went ahead and moved the schema dir from contrib to etc and adapted
> the Makefile to install it.

Now pushed to my Makefile fork after testing it on Win7 (and rebased
again to current master).


Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Factory and User Sound Singles for Waldorf Q+, Q and microQ:
http://Synth.Stromeko.net/Downloads.html#WaldorfSounds

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2012-01-04 17:46 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-21 18:47 Please test the new Makefile Bastien
2011-12-22  1:35 ` Takaaki ISHIKAWA
2011-12-22 10:05   ` Achim Gratz
2011-12-22 15:06     ` Takaaki ISHIKAWA
2011-12-23 16:10       ` Bastien
2011-12-25 10:27       ` ASSI
2011-12-26  4:58         ` Takaaki ISHIKAWA
2011-12-27 10:33           ` Achim Gratz
2012-01-02 21:50     ` Achim Gratz
2012-01-03 18:09       ` Achim Gratz
2012-01-04 17:46         ` Achim Gratz
2011-12-22 10:25 ` Achim Gratz

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.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).