Den 2009-01-21 23:09, skrev Ralf Wildenhues:
* Peter Rosin wrote on Tue, Jan 20, 2009 at 12:34:57AM CET:
Den 2009-01-19 21:35 skrev Ralf Wildenhues:
This looks a bit hackish. We already have a handful of places which we
fixed up in order to avoid line ending issues. This one looks hackish
Den 2009-01-21 22:54, skrev Ralf Wildenhues:
Hi Peter,
* Peter Rosin wrote on Wed, Jan 21, 2009 at 02:26:36AM CET:
This patch together with [1] and [2] will make Run tests with
low max_cmd_len on MSYS/MSVC behave the same as the individual
tests.
The patch fixes a couple more of the /abs/path
Le 19 janv. 09 à 23:38, Ralf Wildenhues a écrit :
Bonjour Ralf!
It is true that I have been disturbed and even annoyed by tab vs.
space in some patches that gave the impression of improper
indentation.
Therefore, for a year or so, I am now using spaces only.
But it is not sufficient for
Den 2009-01-21 23:13, skrev Ralf Wildenhues:
* Charles Wilson wrote on Tue, Jan 20, 2009 at 07:29:52PM CET:
Bob Friesenhahn wrote:
For many years I have had my editor configured to always use
spaces. This ensures WYSIWYG for everyone involved.
Agree 100%. I try to manually match whatever
* Charles Wilson wrote on Wed, Jan 21, 2009 at 08:47:42PM CET:
[...] EVERY separate patchset requires an independent full testsuite
run. Until recently, that was 5 hours of sitting in front of my
computer waiting for popups, while that computer was completely useless
for anything else (100%
On Thu, 22 Jan 2009, Peter Rosin wrote:
Den 2009-01-21 23:13, skrev Ralf Wildenhues:
* Charles Wilson wrote on Tue, Jan 20, 2009 at 07:29:52PM CET:
Bob Friesenhahn wrote:
For many years I have had my editor configured to always use
spaces. This ensures WYSIWYG for everyone
Hi Peter,
* Peter Rosin wrote on Thu, Jan 22, 2009 at 10:57:20AM CET:
Something like the attached?
Yes, with nits below addressed.
Ok, so push to master or pr-msvc-support?
This change is ok for both. However, let's move away from manual
addition of the same change to more than one public
* Bob Friesenhahn wrote on Thu, Jan 22, 2009 at 02:04:07AM CET:
On Wed, 21 Jan 2009, Ralf Wildenhues wrote:
We may need to think about speeding up func_to_host_path, e.g., by not
forking for arguments that don't need conversion, or by converting
several paths with a constant amount of forks.
On Thu, 22 Jan 2009, Tim Rice wrote:
Or maybe it just seems so because those that want a change are those
that will speak up.
I prefer tabs. If I want the text in the 25th column it's just
3 tab keys and start typing. I don't have to count out pressing
the space bar 24 times and then wondering
Den 2009-01-22 21:54 skrev Ralf Wildenhues:
Hi Peter,
* Peter Rosin wrote on Thu, Jan 22, 2009 at 10:57:20AM CET:
Something like the attached?
Yes, with nits below addressed.
Ok, so push to master or pr-msvc-support?
This change is ok for both. However, let's move away from manual
* tests/cwrapper.at: New file.
* Makefile.am: Add tests/cwrapper.at.
---
Ok to push?
Makefile.am |1 +
tests/cwrapper.at | 81 +
2 files changed, 82 insertions(+), 0 deletions(-)
create mode 100644 tests/cwrapper.at
diff --git
Charles Wilson wrote:
The bits removed from this commit are attached as
cygwin-cwrapper-cleanups.patch.
Will start a new thread with these cleanups.
--
Chuck
Ralf Wildenhues wrote:
* Charles Wilson wrote on Wed, Jan 21, 2009 at 08:47:42PM CET:
[...] EVERY separate patchset requires an independent full testsuite
run. Until recently, that was 5 hours of sitting in front of my
computer waiting for popups, while that computer was completely useless
Charles Wilson wrote:
+ sed s/LTCFAGS=.*/' $restrictive_flags'/ $orig_LIBTOOL ./libtool
I've already fixed this typo s/LTCFAGS/LTCFLAGS/
--
Chuck
libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src)
[file scope]: re-organized includes and portability
macros. Avoid oldnames on MINGW32 and MSVC for
setmode/stat/chmod/getcwd/putenv. Declare _putenv on
MINGW32 when -ansi. Use namespaced macro LT_DEBUGWRAPPER.
---
This patch contains the parts
* Peter Rosin wrote on Fri, Jan 23, 2009 at 12:01:00AM CET:
Den 2009-01-22 21:54 skrev Ralf Wildenhues:
1) create, in your own repository, a new branch pr-tests-newline-fixes
containing this one patch, then merge this patch into both
pr-msvc-support and into master, then push the last two (no
* Charles Wilson wrote on Fri, Jan 23, 2009 at 12:36:57AM CET:
* tests/cwrapper.at: New file.
* Makefile.am: Add tests/cwrapper.at.
---
Ok to push?
Yes, with really minor nits:
--- /dev/null
+++ b/tests/cwrapper.at
@@ -0,0 +1,81 @@
+AT_SETUP([wrapper for uninstalled dynamically linked
17 matches
Mail list logo