Re: Latest CEDET on BZR does not compile with emacs 24.1

["Vincent Belaïche" <vincent.b.1@hotmail.fr>]

[-- Attachment #1.1: Type: text/plain, Size: 3147 bytes --]




> Date: Tue, 2 Oct 2012 23:10:31 +0200
> From: eliz@gnu.org
> Subject: Re: [cedet-semantic] Latest CEDET on BZR does not compile with emacs	24.1
> To: vincent.b.1@hotmail.fr
> CC: cedet-semantic@lists.sourceforge.net; deng@randomsample.de; emacs-devel@gnu.org
> 
> > From: Vincent Belaïche <vincent.b.1@hotmail.fr>
> > CC: "cedet-semantic@lists.sourceforge.net"
> > 	<cedet-semantic@lists.sourceforge.net>, emacs-devel <emacs-devel@gnu.org>

[... SNIP ...]> 
> I suggest not to use MSYS at all.  
Is that a general recommendation --- i.e. do not build EMACS with MSYS --- or is that a recommendation for CEDET only. It happens that I could not compile EMACS with MSYS for some time --- is the MSYS port abandoned ?
> If the original CEDET Makefile's
> cannot be changed to not require a Unixy shell, 
The only MSWindows port of GNUMake which I know is the MSYS one. Cygwin is not really a Windows port, it is rather some Unixy environment emulation under MSWindows.
I tried the Makefile with dmake and here is what it does:
C:\Programme\GNU\installation\cedet-install\cedet>dmake EMACS=c:\Programme\GNU\emacs-24.1\bin\emacs.exe 
dmake EMACS=c:\Programme\GNU\emacs-24.1\bin\emacs.exe 
dmake.exe:  makefile:  line 46:  Error: -- Missing targets or attributes in rule

i.e. it seems that dmake is less advanced than GNU Make and does not support addsufix function. 
> write a separate
> Windows specific Makefile with equivalent commands, and be done with
> that.  
I am afraid that the trick which I did is tremendously simpler to do and to maintain than what I understood that you are suggesting. I am not going to re-write all the CEDET Makefiles just to use an MSWindows make engine. I've  read from David that those Makefiles are generated by EDE, so if ever there was something to do, it would be better to upgrade EDE to generates those MSWindows makefile, or anything working well under MSWindows without the need for a Unixy shell (e.g. ant build.xml files). Anyhow, CEDET can be built with some lisp only script that will be soon fixed, so even this thing is not needed.
> As a bonus, latest native ports of GNU Make support all the
> functions that you needed to replace.  You can even replace Sed
> commands with equivalent GNU Make functions, if you want.
>
Thank you for the advice, I totally agree with this one, Makefile internal text & path manipulation should be done as much as possible platform independent.I attached a corrected `platform.mak' which does not use sed but only GNU make functions. You are right, sed is a little old fashioned and not always available.
> MSYS is for configuring and building packages that use Autoconf and
> other autotools.  It is not really appropriate for arbitrary Windows
> jobs.  It can be done, of course, as your patch demonstrates, but it
> will be fragile, and requires MSYS to be installed, which is a large
> set of packages.
> 
Well, I have already MSYS installed, so I did not need any extra installation, while using Cygwin was the opposite in my case, that would have meant installing a large package.
Kind regards,   Vincent. 		 	   		  

[-- Attachment #1.2: Type: text/html, Size: 3910 bytes --]

[-- Attachment #2: platform.mak --]
[-- Type: application/octet-stream, Size: 1065 bytes --]

# Platform dependent settings
#
# (C) 2011, 2012 CEDET Developers
#
# This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License as
# published by the Free Software Foundation; either version 2, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful, but
# WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with GNU Emacs; see the file COPYING.  If not, write to the
# Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
# Boston, MA 02110-1301, USA.
ifeq ($(OSTYPE) , msys)
	AP_LOADDEFS = $(subst /,\\,$(shell cd $(dir $(abspath $(LOADDEFS))); echo "$$(pwd -W)/$(notdir $(LOADDEFS))" ))
	AP_LOADDIRS = $(subst /,\,$(shell cd $(LOADDIRS); pwd -W))
else
	AP_LOADDEFS = $(abspath $(LOADDEFS))
	AP_LOADDIRS = $(abspath $(LOADDIRS))
endif


[-- Attachment #3: Type: text/plain, Size: 387 bytes --]

------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev

[-- Attachment #4: Type: text/plain, Size: 173 bytes --]

_______________________________________________
cedet-semantic mailing list
cedet-semantic@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cedet-semantic