Issue 104727 - patch for dmake to properly work if installed in a dir spaces in its name
Summary: patch for dmake to properly work if installed in a dir spaces in its name
Status: CLOSED WONT_FIX
Alias: None
Product: Build Tools
Classification: Code
Component: dmake (show other issues)
Version: OOo 1.0.0
Hardware: Unknown All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: hjs
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-02 16:51 UTC by kmx
Modified: 2017-05-20 09:37 UTC (History)
4 users (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description kmx 2009-09-02 16:51:40 UTC
Hi,

please find a patch for dmake utility from here:
http://svn.ali.as/cpan/users/kmx/dmake_patch/dmake_svn-rev275670_patch.diff

The patch contains fixes for this:

1) dmake is not able even start if it is installed in a directory containing
spaces in directory name.
- I have to fix dmake itself (4-line patch of expand.c)
- but also startup/*.mk scripts (I have fixed just Windows related files)
- you might want to patch other (e.g. unix) *.mk files in a similar way

2) I have added a build batch for mingw32 enviroment - see newly added
winnt/mingw/build.cmd + winnt/mingw/config.h

Please consider the patch, it would be nice if you can accept it an commit in
your SVN as we are using dmake in strawberry perl project
(http://www.strawberryperl.com) and do not want to keep any sort of unofficial
dmake patches.

Thanks

--
kmx
Comment 1 kmx 2009-10-20 09:20:00 UTC
Hi,

could you please give me some feedback to the proposed patch. If you do not like
anything included in my patch I'll be happy to fix it or help you in other way.

The most important part is a patch for expand.c + startup/*.mk files as it is a
blocker for using dmake from directory which name contains spaces.

Thanks.

--
kmx
Comment 2 hjs 2009-10-20 17:28:26 UTC
sorry, must have missed that one completely.

the patch look fine on first glance. i'll have a closer look soon. don't
hesitate to ping me as dmake changes aren't my current focus.

meanwhile please check out
http://wiki.services.openoffice.org/wiki/Contributing_Patches , especially the
first paragraph.
Comment 3 kmx 2009-10-23 13:37:28 UTC
> meanwhile please check out
> http://wiki.services.openoffice.org/wiki/Contributing_Patches , 
> especially the first paragraph.
Well, you mean I have to sign the agreement?

To be honest I do not have any ambitions to participate in OpenOffice
development process. I have just this veeery simple patch to dmake tool and I
promise this is my last patch, I am not gonna submit any other.

If you consider anything in my very simple patch a intellectual property I
declare that I have coded it myself, it is absolutely patent-free and you or
anybody else can release it under whatever licence you want.

We (strawberry perl project) are going to do a major binaries rebuild for
January-2010 release (incl. dmake) and it would be nice to have at least the
patch for expand.c + startup/*.mk in the latest SVN trunk version. 

Thanks.

--
kmx
Comment 4 kmx 2009-11-08 21:34:23 UTC
I have updated the previously submitted patch a little bit - see:
http://svn.ali.as/cpan/users/kmx/dmake_patch/dmake_svn-rev277396_patch.diff

The previous (not so good) patch removed.

--
kmx
Comment 5 kmx 2009-11-27 07:32:41 UTC
Bad news, my patch does not work well with the following Makefile:

#########
LIST1	= a.a ../b.b ./c.c ..\d.d .\e.e /f.f \g.g c:/h/h.h c:\i\i.i
LIST2	= $(LIST1:f)

all:
	echo "LIST2=$(LIST2)"
#########

At the moment I have no idea how to fix it.

I would appreciate any help.

--
kmx
Comment 6 Pedro Giffuni 2012-10-16 04:10:54 UTC
I copied this to the new project "home":
http://code.google.com/a/apache-extras.org/p/dmake/