USB and Serial Port add-ons added

[Michael Pfeiffer]

Hi,

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).

BTW it seems BeOS R5 supports one connected USB printer only. Can somebody
verify this?

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_rule + 00000171
fcfe5278 8001138c compile_rule + 000000b8
fcfe52cc 8001160a compile_rules + 0000007e
fcfe52e4 80009a61 parse_file + 00000071
fcfe5328 80010fde compile_include + 000000d2

Philippe or whoever reads this, please check that the checked in
source code builds without errors
(current/src/add-ons/print/transports/usb_port, serial_port, parallel_port).

TIA
- Michael

[Philippe Houdoin]

Hi Michael,

Post by Michael Pfeiffer
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
;-)

Post by Michael Pfeiffer
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.

Post by Michael Pfeiffer
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).
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

Post by Michael Pfeiffer
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.

[Michael Pfeiffer]

Hi Philippe,

Post by Philippe Houdoin
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...

I agree, but this is a R2 thing.

[Jam crash]

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

No.

Post by Philippe Houdoin
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 :-)

You're right, it's now in the repository.

Post by Philippe Houdoin
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

Please do so.

- Michael