Re: X41 ACPI suspend hints

2010-08-11 Thread J.C. Roberts
On Thu, 12 Aug 2010 00:08:57 +0200 David Coppa wrote: > > On Wed, 11 Aug 2010, Mark Kettenis wrote: > > > > Date: Tue, 10 Aug 2010 23:15:32 +0200 > > > From: Mike Belopuhov > > > > > > Hi, > > > > > > I have a Thinkpad X41 which doesn't do ACPI suspend properly > > > (it immediately wakes up a

Re: trivial diff for plus.html

2010-08-10 Thread J.C. Roberts
On Tue, 10 Aug 2010 05:56:02 -0700 "J.C. Roberts" wrote: > > a trivial fix of cd(4) versus cdio(1), > but this diff will probably break due to line length > of the input file exceeding 998 characters. Ya, I know there's no line length limit specified in RFC822, but

trivial diff for plus.html

2010-08-10 Thread J.C. Roberts
a trivial fix of cd(4) versus cdio(1), but this diff will probably break due to line length of the input file exceeding 998 characters. Index: plus.html === RCS file: /cvs/www/plus.html,v retrieving revision 1.1174 diff -u -r1.1174 p

Re: suspend code for all !intagp agp drivers. NEEDS TESTING

2010-08-07 Thread J.C. Roberts
On Fri, 6 Aug 2010 14:34:22 +0100 Owain Ainsworth wrote: > > Other than intagp, pretty much every agp driver is almost identical, > with only small details differing. Unlike intagp, the ptes are not in > a BAR, but in dma memory. so this diff just repeats every register > setting that we do on att

4K Sector Disks

2010-06-28 Thread J.C. Roberts
dlg, It took me weeks and a few failed attempts with various disk manufacturers, but it's done, and we have victory! The value from the modified atactl output for reg 106: 4000 Finally it seems we have a disk that is properly showing us 4k sectors rather than lying. I kind of guessed this disk m

Re: defining ports LOCKDIR

2010-06-16 Thread J.C. Roberts
On Wed, 16 Jun 2010 13:38:12 +0200 Marc Espie wrote: > > Historically, I did leave LOCKDIR empty in bsd.port.mk. > Now, I'm wondering if I should define it to something such as > /tmp/portslocks ? > > My assumption was that it would be difficult to be certain that > dotlocks could be created anyw

PCIID and DRM for Mobility Radeon HD 4870

2010-06-15 Thread J.C. Roberts
Added pci id and drm support for Mobility Radeon HD 4870 Tested on amd64 with June 10th snapshot (gcc4) NOTE: Though it works very well, I'm not entirely convinced the "CHIP_RV770" part is correct in radeon_drv.c but it seems close enough for the moment. I'm probably just confusing myself with the

Re: (another) Intel driver change needs testing.

2010-05-26 Thread J.C. Roberts
On Tue, 25 May 2010 21:21:49 -0700 Philip Guenther wrote: > > On Sun, May 23, 2010 at 10:57 PM, J.C. Roberts > wrote: ... > > I found the problem. The 'xorg' vendor branch was treated as 'HEAD' > > for a while, but subsequently moving 'HEAD'

BUG: need to add 4.7 to man.cgi

2010-05-26 Thread J.C. Roberts
We need a '4.7' entry for man.cgi since at present, when you ask it for current, you get 4.7. -- The OpenBSD Journal - http://www.undeadly.org

Re: (another) Intel driver change needs testing.

2010-05-23 Thread J.C. Roberts
On Sun, 23 May 2010 15:53:57 -0700 "J.C. Roberts" wrote: > Since it seems I'm the first to find a problem with the '-D' flag, I'm > testing to see if it is being caused by my using the ISO date_spec > format by doing a similar checkout with the "Intern

Re: (another) Intel driver change needs testing.

2010-05-23 Thread J.C. Roberts
On Sun, 23 May 2010 17:19:03 +0100 Owain Ainsworth wrote: > On Sun, May 23, 2010 at 10:15:21AM -0600, Ted Roby wrote: > > On Sun, May 23, 2010 at 9:55 AM, Owain Ainsworth > > wrote: > > > > > > > > > > > > > > You mean that the build failed? or that the corruption happened? > > > > > > > > I mean

Re: New intel X driver requires testing.

2010-05-21 Thread J.C. Roberts
On Sun, 25 Apr 2010 14:19:04 -0700 "J.C. Roberts" wrote: > On Sun, 11 Apr 2010 18:47:45 +0100 Owain Ainsworth > wrote: > > > The tarball that may be found at > > http://xenocara.org/intel-current.tgz contains an update to the > > intel 2.9.1 driver

Re: (another) Intel driver change needs testing.

2010-05-19 Thread J.C. Roberts
On Wed, 19 May 2010 09:36:14 +0200 David Coppa wrote: > > That's good news! I don't have the "experimental" mesa patch, but > > if you want me to test it, mail it to me off list. > > I've spoken too soon! > The problem now has changed, but it's still here. > > Now the screen goes blank but not t

Re: (another) Intel driver change needs testing.

2010-05-19 Thread J.C. Roberts
On Wed, 19 May 2010 20:10:49 +0200 (CEST) Mark Kettenis wrote: > > When testing code controlling video/graphics hardware, you should > > power off the system after hitting an error. The reason is simple; > > the hardware/memory is no longer in a know state, so you no longer > > know what you are t

Re: (another) Intel driver change needs testing.

2010-05-19 Thread J.C. Roberts
On Wed, 19 May 2010 09:36:14 +0200 David Coppa wrote: > > > The occasional/intermittent screen corruption bug seems to have > > > disappeared for me with latest xenocara -current + the > > > "experimental" Mesa 7.8 update. > > > > That's good news! I don't have the "experimental" mesa patch, but

Re: move klist to process

2010-05-18 Thread J.C. Roberts
On Mon, 17 May 2010 21:08:05 -0400 (EDT) Ted Unangst wrote: > This moves the knote list to struct process for better process > semantics with rthreads. It should have little noticeable effect > otherwise, but I've been wrong about simpler diffs before > It has no noticeable effect on i386,

Re: (another) Intel driver change needs testing.

2010-05-18 Thread J.C. Roberts
On Tue, 18 May 2010 07:40:23 -0700 "J.C. Roberts" wrote: > It seems to become enabled, but really doesn't do anything. > > (II) intel(0): DPMS enabled > (==) intel(0): Intel XvMC decoder disabled copy

Re: (another) Intel driver change needs testing.

2010-05-18 Thread J.C. Roberts
On Tue, 18 May 2010 16:52:11 +0200 David Coppa wrote: > On Tue, May 18, 2010 at 4:40 PM, J.C. Roberts > wrote: > > > Whether enabled or disabled in xorg.conf, (so far) it seems to have > > no effect on the occasional/intermittent screen corruption bug seen > > by

Re: (another) Intel driver change needs testing.

2010-05-18 Thread J.C. Roberts
On Mon, 17 May 2010 22:16:54 +0100 Owain Ainsworth wrote: > For those of you new here, or termnally adsent minded, instructions > follow: > You've got be careful with those "termnally adsent" folks. ;) With the 845G and a minimal xorg.conf (only "NoTrapSignals") I get the expected: (II

Re: typo sys/conf/GENERIC

2010-05-12 Thread J.C. Roberts
On Wed, 12 May 2010 14:36:08 +0200 Joachim Schipper wrote: > On Wed, May 12, 2010 at 03:01:50AM -0700, J.C. Roberts wrote: > > minor typo > > It's meant to be "interface"; following ifconfig(8), one could say > 'if' makes more sense than 'i/f&#x

typo sys/conf/GENERIC

2010-05-12 Thread J.C. Roberts
minor typo Index: GENERIC === RCS file: /cvs/src/sys/conf/GENERIC,v retrieving revision 1.156 diff -N -u -p GENERIC --- GENERIC 7 May 2010 13:16:18 - 1.156 +++ GENERIC 12 May 2010 09:57:26 - @@ -112,7 +112,7 @@ p

no faith ;)

2010-05-06 Thread J.C. Roberts
Neither mpe(4) nor faith(4) are presently hooked up to /src/sys/conf/GENERIC build so this might be a bit early. mpe(4) has been documented in ifconfig(8) already, but there's no mention of faith(4) in there. Index: ifconfig.8 === RCS

Re: Re drm for r600 and r700

2010-04-29 Thread J.C. Roberts
On Thu, 29 Apr 2010 11:12:38 +0200 Azwaw OUSADOU wrote: > I tried drm with radeonhd driver and radeon driver. It's doesn't > work. Xorg crash at start. You haven't taken the time to figure out the difference between the 'radeon' driver and the 'radeonhd' driver. Search the mail list archives.

Re: diff to fix/clean up ep(4) ioctl handler.

2010-04-26 Thread J.C. Roberts
On Tue, 20 Apr 2010 23:22:21 -0400 Brad wrote: > A diff for the ep(4) 3Com EtherLink III driver to clean up the ioctl > handler code a bit and eliminate unnecessary resets when > reconfiguring IP addresses. First I tried to establish a baseline *without* your patch only to find the 3C595-TX ep(4

Re: New intel X driver requires testing.

2010-04-25 Thread J.C. Roberts
On Sun, 11 Apr 2010 18:47:45 +0100 Owain Ainsworth wrote: > The tarball that may be found at http://xenocara.org/intel-current.tgz > contains an update to the intel 2.9.1 driver (the last one that > supported userland modesetting) with a load of backports for bugfixes > and performance improvemen

Re: bge(4) diff needs testing.

2010-04-23 Thread J.C. Roberts
On Wed, 21 Apr 2010 20:59:46 -0400 Brad wrote: > > Which ones should people tests? > > The bge(4), msk(4), ne(4) and ep(4) diffs. > > bge(4) and msk(4) I have and have tested those respective diffs. > I have no ne(4) or ep(4) hardware, the changes are mechanical but > still need testing. I did

Re: ifconfig.8 trunk section

2010-04-23 Thread J.C. Roberts
On Sun, 11 Apr 2010 16:03:42 +0059 Jason McIntyre wrote: > - it should be "an mpe device", not "a mpe device" (i'll fix the one > in mpe.4) The determining factor for the correct use of the indefinite articles "a" and "an" is based on pronunciation. If the word starts with a classic vowel sound,

Re: New intel X driver requires testing.

2010-04-23 Thread J.C. Roberts
On Thu, 22 Apr 2010 21:15:26 +0200 Matthieu Herrb wrote: > > Also, virtual terminal > > switching has been problematic over here with the new driver, so > > testing it thoroughly on other systems is wise. Over here, to hit > > the bug you'll have to start x, then virtual terminal switch a few > >

Re: New intel X driver requires testing.

2010-04-22 Thread J.C. Roberts
I've noticed an interesting difference between the new driver, and the old/existing driver in the April 15th i386 snapshot (dmesg below). With the old/existing intel driver in the snapshot, my 845G uses EXA acceleration, and Xv doesn't work. Attempting to force the driver to use UXA acceleration v

Re: New intel X driver requires testing.

2010-04-22 Thread J.C. Roberts
On Thu, 22 Apr 2010 09:04:06 +0200 Joachim Schipper wrote: > This machine, a Thinkpad SL510, worked fine with the Xenocara/the > driver from a snaphot; I compiled Xenocara (not just the driver - I > got compile errors) for the newer version. (So note that I've never > tried this Xenocara with the

Re: Source Overview

2010-04-19 Thread J.C. Roberts
On Mon, 19 Apr 2010 16:57:38 +0200 "Bret S. Lambert" wrote: > src/sys/ > kern/ <- generic stuffs (signals, scheduling, vnodes, > syscalls) net/<- generic net stuffs (interface handling, pf, > routing) netinet{,6}/ <- IPv{4,6} stuffs > net*/ <- non-IP network stuffs >

Re: New intel X driver requires testing.

2010-04-19 Thread J.C. Roberts
On Sun, 11 Apr 2010 18:47:45 +0100 Owain Ainsworth wrote: > The tarball that may be found at http://xenocara.org/intel-current.tgz > contains an update to the intel 2.9.1 driver (the last one that > supported userland modesetting) with a load of backports for bugfixes > and performance improvemen

Re: testers needed for auich(4)

2010-04-02 Thread J.C. Roberts
On Fri, 2 Apr 2010 22:45:12 +0200 Alexandre Ratchov wrote: > > A short summary of the changes: > > > > - According to specification AUICH_RR may only be set after DMA > > is halted (AUICH_DCH is 0 in AUICH_STS). To accomplish this I > > revived auich_halt_pipe(); > > - auich_calibrate() did

Re: pmemrange recommit (ok request)

2010-03-28 Thread J.C. Roberts
On Thu, 25 Mar 2010 07:50:15 + "Owain G. ainsworth" wrote: > Ifit uses dma (grep the driver for isa_dmastart) then it would be > useful. current tests imply it is ok though. > -- Unfortunately, the ISA network cards I have here 3C509B (ep(4) isapnp) don't use isa_dmastart. Well, more accurat

Re: fdisk(8) man page diff

2010-03-25 Thread J.C. Roberts
On Thu, 25 Mar 2010 19:22:21 -0400 Anders Langworthy wrote: > The following patch provides some minor grammar edits, adds some > markup, and documents a few features that aren't mentioned in the > existing man page. > > This is my first effort at submitting a diff to OpenBSD (or any other > proj

Re: pmemrange recommit (ok request)

2010-03-24 Thread J.C. Roberts
On Wed, 24 Mar 2010 16:53:24 +0100 Ariane van der Steldt wrote: > If you applied oga's diff prior to mine, ignoring the rejection in > sys/arch/i386/isa/isa_machdep.c is the right course of action. > Otherwise repatching with oga's diff (as posted) and mine (below, > without the sneaky change) is

Re: CVS: cvs.openbsd.org: www

2010-03-14 Thread J.C. Roberts
On Sat, 13 Mar 2010 16:19:09 -0700 (MST) Theo de Raadt wrote: > CVSROOT: /cvs > Module name: www > Changes by: dera...@cvs.openbsd.org 2010/03/13 16:19:09 > > Added files: > images : 47song.jpg Superfish.jpg cd47-s.gif >openbsdcover47_cover.gif post

Re: suggested patch to httpd.conf in base

2010-03-12 Thread J.C. Roberts
On Fri, 12 Mar 2010 16:17:51 -0700 Bob Beck wrote: > Off is off. don't make it where you have to turn 8 knobs to turn > something on. because you wanted it "more off". Alternatively, you could make the user turn 8 knobs to turn something "moron" ;) (sorry, couldn't resist)

Re: faq14.html

2010-03-12 Thread J.C. Roberts
Now that the big changes have been seen together so they make more sense, I've broken the changes into smaller, more manageable chunks. As usual, only Nick will the stripped attachments, but you can access them from my server. The diffs need to be applied in order. I started with the current (as o

Re: faq14.html

2010-03-10 Thread J.C. Roberts
Thanks to Tymon for catching a missing word and a missing asterisk. Changes made on the hosted page, and new diff http://www.designtools.org/faq14.html http://www.designtools.org/faq14.html.diff3 jon

Re: faq14.html

2010-03-10 Thread J.C. Roberts
On Mon, 08 Mar 2010 07:31:55 -0500 Nick Holland wrote: > J.C. Roberts wrote: > > Initially, I just wanted to fix the "use create" typo, but decided > > to make it shorter, more accurate, and more clear in the process. > > committed. > btw: this is a good type

Re: SiS 7002 lockup using USB2.0 'legacy' mode

2010-03-07 Thread J.C. Roberts
On Sun, 7 Mar 2010 22:31:18 +0100 Christopher Zimmermann wrote: > ok, you misunderstood me. My BIOS needs 'legacy' to be turned on to > use the USB-keyboard. Yep, you weren't clear on "legacy" as meaning "Legacy USB Support" but I was able to dig up a possibly similar/correct manual. According

faq14.html

2010-03-07 Thread J.C. Roberts
Initially, I just wanted to fix the "use create" typo, but decided to make it shorter, more accurate, and more clear in the process. Index: faq14.html === RCS file: /cvs/www/faq/faq14.html,v retrieving revision 1.198 diff -N -u -p f

Re: SiS 7002 lockup using USB2.0 'legacy' mode

2010-03-07 Thread J.C. Roberts
On Sun, 7 Mar 2010 17:24:07 +0100 Christopher Zimmermann wrote: > Hi, > > I get a complete lock-up while booting during initialisation of my > SIS7002 EHCI controller. > > When enabling USB2.0 support, but disabling 'legacy' mode everything > is fine. I just can't use my usb keyboard to talk

Re: installboot.8

2010-03-05 Thread J.C. Roberts
On Fri, 5 Mar 2010 06:44:41 +0001 Jason McIntyre wrote: > On Thu, Mar 04, 2010 at 06:56:30PM -0800, J.C. Roberts wrote: > > The deleted statement is not true for i386 (probably "no longer" > > true) and similar statements are in the BUGS section (rather than > > th

installboot.8

2010-03-04 Thread J.C. Roberts
The deleted statement is not true for i386 (probably "no longer" true) and similar statements are in the BUGS section (rather than the main body) of the installboot.8 man pages for other archs. The statements about securelevel requirements in the other archs are often somewhat different than i386.

/arch/i386/stand/Makefile

2010-03-04 Thread J.C. Roberts
There is no '-r' flag for disklabel. Index: Makefile === RCS file: /cvs/src/sys/arch/i386/stand/Makefile,v retrieving revision 1.15 diff -u Makefile --- Makefile25 Nov 2007 18:25:31 - 1.15 +++ Makefile4 Mar 2010 21:5

Re: ports.7

2010-03-04 Thread J.C. Roberts
On Thu, 4 Mar 2010 15:21:34 +0100 Marc Espie wrote: > On Thu, Mar 04, 2010 at 06:14:52AM -0800, J.C. Roberts wrote: > > On Thu, 4 Mar 2010 09:38:12 +0100 Marc Espie > > wrote: > > > > > On Wed, Mar 03, 2010 at 04:46:18PM -0800, J.C. Roberts wrote: > > >

Re: ports.7

2010-03-04 Thread J.C. Roberts
On Thu, 4 Mar 2010 09:38:12 +0100 Marc Espie wrote: > On Wed, Mar 03, 2010 at 04:46:18PM -0800, J.C. Roberts wrote: > > Boolean variables should be set to Yes instead of simply > > being defined, for uniformity and future compatibility. > > This takes time to finish ch

ports.7

2010-03-03 Thread J.C. Roberts
The ports(7) man page reads: --- and use the update target with FORCE_UPDATE set to `Yes' for the build. ... PORT VARIABLES These can be changed in the environment, or in /etc/mk.conf for persistence

Re: mistake in fsck_ffs.8

2010-03-03 Thread J.C. Roberts
On Wed, 3 Mar 2010 13:07:03 + Jason McIntyre wrote: > On Wed, Mar 03, 2010 at 01:24:09PM +0100, Otto Moerbeek wrote: > > > > Ok, with tweaks. I don't it is neccesary to repeat we are dealing > > with FFS, nor is the e.g. useful, imo. jmc, do you agree? > > > > yes, i'm fine with this lates

Re: mistake in fsck_ffs.8

2010-03-03 Thread J.C. Roberts
On Wed, 3 Mar 2010 12:11:43 +0100 Otto Moerbeek wrote: > +.It Fl p > +Enter preen mode: > +.Nm > +will check all file systems listed on the command line > +and will make minor repairs without > +human intervention. > +Any major problems will cause > +.Nm > +to exit with a non-zero exit code, > +s

Re: mistake in fsck_ffs.8

2010-03-03 Thread J.C. Roberts
On Wed, 3 Mar 2010 11:36:28 +0100 Otto Moerbeek wrote: > > If fsck_ffs is *supposed* to read /etc/fstab like fsck does, then > > the problem is in the code. > > fsck_ffs is not supposed to read fstab, so this line should be gone. > Diff below. > > > > > Also the `-p` option (preen) is not in t

mistake in fsck_ffs.8

2010-03-03 Thread J.C. Roberts
Unfortunately, the following statement is not true: If no file systems are given to fsck_ffs then a default list of file sys- tems is read from the file /etc/fstab. To test, boot single user and run # fsck_ffs -fp If fsck_ffs is *supposed* to read /etc/fstab like fsck doe

Re: httpd mime.types OpenDocument Format

2010-03-02 Thread J.C. Roberts
On Tue, 23 Feb 2010 17:45:43 +0200 (EET) Lars Nooden wrote: > Adding the IANA mime types for the OpenDocument Format, with file > name extensions. The previous diff was missing the extensiosn. > For references regarding this open standard, see > > a) ISO/IEC 26300, The OpenDocument Format > >

Re: [doc-patch] ksh.1

2010-01-12 Thread J.C. Roberts
On Mon, 11 Jan 2010 13:13:44 -0800 Philip Guenther wrote: > On Mon, Jan 11, 2010 at 1:01 AM, J.C. Roberts > wrote: > > Below is a "test case" ksh shell script to show examples of very > > minor omissions in the ksh(1) man page regarding testing for zero > > l

[doc-patch] ksh.1

2010-01-11 Thread J.C. Roberts
2) string is *not* empty (Should Be TRUE)\n"; printf "\t'if [[ -n string ]]'\n"; if [[ -n $foo ]]; then printf "\t%s\n" 'PASS: Returned TRUE'; else printf "\t%s\n" 'FAIL: Returned FALSE -Noted under [ expression ]'; fi printf "--\n"; printf "#8.C.1) string is empty (Should Be FALSE)\n"; printf "\t'if [ -z string ]'\n"; if [ -z $foo ]; then printf "\t%s\n" 'FAIL: Returned TRUE'; else printf "\t%s\n" 'PASS: Returned FALSE ***FIXME error***'; fi printf "--\n"; printf "#8.C.2) string is empty (Should Be FALSE)\n"; printf "\t'if [[ -z string ]]'\n"; if [[ -z $foo ]]; then printf "\t%s\n" 'FAIL: Returned TRUE'; else printf "\t%s\n" 'PASS: Returned FALSE'; fi printf "--\n"; # @@ foo='-z that' printf "\n**\n"; printf "%s%s\n" 'Test #9: WITH $foo DEFINED unary: ' "foo='-z that'"; printf "--\n"; printf "#9.A.1) string has non-zero length (Should Be TRUE)\n"; printf "\t'if [ string ]'\n"; if [ $foo ]; then printf "\t%s\n" 'PASS: Returned TRUE'; else printf "\t%s\n" 'FAIL: Returned FALSE -Noted Under [ expression ]'; fi printf "--\n"; printf "#9.A.2) string has non-zero length (Should Be TRUE)\n"; printf "\t'if [[ string ]]'\n"; printf "\tERROR -Noted under [[ expression ]]\n"; # if [[ $foo ]]; # then # printf "\t%s\n" 'PASS: Returned TRUE'; # else # printf "\t%s\n" 'FAIL: Returned FALSE'; # fi printf "--\n"; printf "#9.B.1) string is *not* empty (Should Be TRUE)\n"; printf "\t'if [ -n string ]'\n"; if [ -n $foo ]; then printf "\t%s\n" 'PASS: Returned TRUE'; else printf "\t%s\n" 'FAIL: Returned FALSE -Noted under [ expression ]'; fi printf "--\n"; printf "#9.B.2) string is *not* empty (Should Be TRUE)\n"; printf "\t'if [[ -n string ]]'\n"; if [[ -n $foo ]]; then printf "\t%s\n" 'PASS: Returned TRUE'; else printf "\t%s\n" 'FAIL: Returned FALSE -Noted under [ expression ]'; fi printf "--\n"; printf "#9.C.1) string is empty (Should Be FALSE)\n"; printf "\t'if [ -z string ]'\n"; if [ -z $foo ]; then printf "\t%s\n" 'FAIL: Returned TRUE'; else printf "\t%s\n" 'PASS: Returned FALSE ***FIXME error***'; fi printf "--\n"; printf "#9.C.2) string is empty (Should Be FALSE)\n"; printf "\t'if [[ -z string ]]'\n"; if [[ -z $foo ]]; then printf "\t%s\n" 'FAIL: Returned TRUE'; else printf "\t%s\n" 'PASS: Returned FALSE'; fi printf "--\n"; exit 0 --- END KSH Script - --- START ksh.1 patch - Index: ksh.1 === RCS file: /cvs/src/bin/ksh/ksh.1,v retrieving revision 1.129 diff -u ksh.1 --- ksh.1 28 May 2009 06:09:06 - 1.129 +++ ksh.1 11 Jan 2010 08:21:22 - @@ -761,6 +761,8 @@ .No \&[ Ar str No \&] use .No \&[[ -n Ar str No \&]] . +Similarly, testing for zero length should use +.No \&[[ -z Ar str No \&]] . .It Parameter, command, and arithmetic substitutions are performed as expressions are evaluated and lazy expression evaluation is used for the @@ -4013,6 +4015,35 @@ Use tests like .Dq if \&[ \&"X$foo\&" = Xbar \&] instead. +.Pp +Additionally, testing for zero length with +.Dq if \&[ -z $foo \&] +, or testing for non-zero length with either +.Dq if \&[ -n $foo \&] +, or +.Dq if \&[ $foo \&] +can also fail if parameter +.Dq foo +is +.Dv NULL +, or unset, or contains spaces (i.e.\& +.Ev IFS +characters), or contains newline characters, or contains unary operators like +.Sq Fl n +or +.Sq Fl z +or +.Sq \&! . +If you are not intentionally trying to evaluate multiple elements in a list, +you can avoid these caveats if you use +.Dq if \&[\&[ -z $foo \&]\&] +for testing zero length, and use +.Dq if \&[\&[ -n $foo \&]\&] +for testing non-zero length. As noted above, the use of +.Dq if \&[\&[ $foo \&]\&] +is not valid, so you'll need to use the +.Sq Fl n +operator. .Pp .It Xo .Ic time --- END ksh.1 patch - -- J.C. Roberts

Re: i845 working with intel-2.7.1 on current

2009-05-17 Thread J.C. Roberts
On Sun, 17 May 2009 13:25:46 -0700 "J.C. Roberts" wrote: > The existing caveat of the intel driver (all versions) being greedy > about resources (either with an xorg.conf, and especially without), > is still problematic. With older drivers, you could back off the from

i845 working with intel-2.7.1 on current

2009-05-17 Thread J.C. Roberts
oga@ asked me to do further testing on i845 and get a bug report filed with the intel folks. The in-tree intel-2.4.3 driver *can* somewhat work on i845 without a xorg.conf, but it's buggy (background drawing problems) when it works, and whether or not it even works is intermittent. Before writing