Re: USB and Serial Port add-ons added

Hi Michael,

> have commited the USB Port transport add-on contributed by Andreas 
> Benzler
> and my untested Serial Port add-on (which is a slightly modified copy 
> of the 
> Parallel Port
> add-on).

Woohoo, running to be first team reaching his Final stage=3F
;-)
 
> BTW it seems BeOS R5 supports one connected USB printer only. Can 
> somebody
> verify this=3F

Not me, as I don't have an USB printer, so can't talk for two usb 
printers even!
However, I'm not supprised, as I don't see how printer driver can pass 
USB transport 
which printer device (/dev/printer/usb/*) it should open...
I think Dano updated USB transport addon offer a better support... 

I guess instead if these hardcoded transport=5Faddress sub-menus in the 
Add Printer setup dialog, we may have some mechanism letting each 
transport addon display and handle his own configure view...

But, being R5-compatible is great already here.
 
> I have a more than a have year old copy of the cvs repository and 
> since I 
> updated to the latest version of jam, Jamfile and Jambase in the root 
> directory, 
> I can not build anything, because jam crashes
> 
> I get this
> warning: unknown rule SystemMain 
> 4 times
> and then jam crashes (it seems with a stack overflow).
> The stack trace repeats these lines some thousand times:
> fcfe5254   8001153d  evaluate=5Frule + 00000171
> fcfe5278   8001138c  compile=5Frule + 000000b8
> fcfe52cc   8001160a  compile=5Frules + 0000007e
> fcfe52e4   80009a61  parse=5Ffile + 00000071
> fcfe5328   80010fde  compile=5Finclude + 000000d2 

Did you post this crash report to our build system guru, Ingo=3F

> Philippe or whoever reads this, please check that the checked in 
> source code builds without errors 
> (current/src/add-ons/print/transports/usb=5Fport, serial=5Fport, 
> parallel=5Fport).

It don't, sorry:

$ pwd
/boot/home/Desktop/openbeos/current/src/add-ons/print/transports 
$ jam Serial\ Port
[...]
MkDir1 ../../../../objects/x86.R1/add-ons/print/transports/serial=5Fport
C++ ../../../../objects/x86.R1/add-ons/print/transports/serial=5Fport/
print=5Ftransport.o
/boot/home/Desktop/openbeos/current/src/add-ons/print/transports/
serial=5Fport/print=5Ftransport.cpp:38: TransportAddOn.h: No such file or 
directory
[...]

Same for "USB Port" target

I search this TransportAddOn.h in our CVS and in CVS history, without 
any success.
Guess someone forgot to commit his local headers/private/print/
TransportAddOn.h :-)

BTW, "transport" add-on may not be only a print concept, may I suggest 
we rename it
into PrintTransportAddOn.[cpp|h] instead=3F. 
I volunteer to do it, as soon as I get my hand on your TransportAddOn.h 
(or, if you want, I can write one from scratch...).

-Philippe


--
Fortune Cookie Says:

You will be attacked by a beast who has the body of a wolf, the tail of
a lion, and the face of Donald Duck.