I have now run every test script <= t9001 using the current 'master' on
the SunOS 5.8 box available to me. It _mostly_ passed. Following are
three patches:
1/3: git_config_*: don't assume we are parsing a config file
This is an actual minor bug which was covered by most platforms'
ability to print (null) for NULL. I posted this earlier, but with
a typo.
2/3: t3404: use configured shell instead of /bin/sh
This is a problem purely in the test script, not git itself.
3/3: diff: fix java funcname pattern for solaris
This is a Solaris-specific bug in git.
With these fixes, everything passes for me with a few exceptions:
- Sun's diff doesn't understand "-u". I was able to use GNU diff.
Since comparing actual and expected output is so common, we could
potentially abstract this with a "test_cmp()" function and use
something platform specific. It's probably not worth the trouble, as
it impacts only the test suite, and only on systems with a totally
broken diff.
- t4020 fails without gnu grep, as it requires "grep -a"
- t4024 fails unless /usr/xpg4/bin is in your path, as the regular
version of 'expr' is unhappy with us (I didn't investigate too
closely)
- t4118, t4200, t8001, and t8002 fail unless you have GNU sed. The
xpg4 sed barfs if there's no newline at the end of input (and this
is specifically part of these tests)
- I couldn't personally run t5502, t5505, t5512, t5520, or t9001, all
of which require the Git.pm module; the version of perl on this
machine is old (5.005_02) and complained about the 'our' keyword
- t7505 uses a hook which hard-codes /bin/sh but uses syntax that
the Solaris /bin/sh doesn't like. Maybe hooks should get their
$SHELL value replaced?
- t3900 mostly passes, but fails to correctly convert between EUCJP
and ISO-2022-JP (in either direction). I didn't investigate very
far.
- git-filter-branch invokes "sh" for the commit filter, which invokes
the broken Solaris /bin/sh. This isn't wrong, exactly, but it might
make more sense to invoke $SHELL.
Now I must go gouge out my eyes, so that I may never see the output of
Solaris /bin/sh ever again.
-Peff
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
| Linus Torvalds | Re: [GIT]: Networking |
| Mark Lord | Re: [Bug #10391] 2.6.25-rc7/8: Another resume regression |
| David Miller | Slow DOWN, please!!! |
| James Bottomley | [Ksummit-2008-discuss] Fixing the Kernel Janitors project |
git: | |
| Ittay Dror | detecting rename->commit->modify->commit |
| Peter Stahlir | Git as a filesystem |
| Junio C Hamano | [ANNOUNCE] GIT 1.5.4 |
| Andreas Hildebrandt | CVS-$Id:$ replacement in git? |
| Gilles Chehade | Re: Real men don't attack straw men |
| carlopmart | About Xen: maybe a reiterative question but .. |
| Christophe Rioux | OpenBSD as host for VMWare Server |
| Kevin Neff | Patching a SSH 'Weakness' |
| David Miller | Re: [BUG] New Kernel Bugs |
| Pavel Emelyanov | [PATCH 4/5] Mark the setup_net as __net_init |
| Chris Snook | [PATCH 0/4] atlx: add atl2 support |
| Jeff Garzik | Re: [RFC][PATCH 1/1] cxgb3i: cxgb3 iSCSI initiator |
| high memory | 4 hours ago | Linux kernel |
| semaphore access speed | 7 hours ago | Applications and Utilities |
| the kernel how to power off the machine | 8 hours ago | Linux kernel |
| Easter Eggs in windows XP | 11 hours ago | Windows |
| Shared swap partition | 12 hours ago | Linux general |
| Root password | 12 hours ago | Linux general |
| Where/when DNOTIFY is used? | 14 hours ago | Linux kernel |
| How to convert Linux Kernel built-in module into a loadable module | 16 hours ago | Linux kernel |
| Linux 2.6.24 and I/O schedulers | 17 hours ago | Linux kernel |
| USB Driver -- Interrupt Polling -- A Little Help Please | 22 hours ago | Linux general |
