vBulletin Search Engine Optimization
| |||||||
| Register | FAQ | Members List | Calendar | Search | Today's Posts | Mark Forums Read |
| ||||
| On Mon, Mar 19, 2007 at 12:11:43AM -0400, Daniel Dickman wrote: > Not sure if this is useful or not, but this patch moves sys/scsi to > sys/dev/scsipi. The rationale for doing this are as follows: from my point of view i dont see an advantage to this. > 1) sys/dev/ is a more logical place for this code. that would be true if this was new code. the reality is this is old code with a lot of changes, and changes that we want to be easily visible. moving it in the tree in this fashion makes it harder to see that history, which in turn makes it harder to work on. i dont see the logic in making this harder to work on. > 2) This patch allows us to remove 3 instances of "#ifdef __OpenBSD__" we try to avoid putting multiple changes into the same diff. can you send this change again separately? > 3) This patch puts the code in the same place as NetBSD. why? > The attached patch fixes up all references to sys/scsi to point to > sys/dev/scsipi. After applying the patch, the contents of sys/scsi > need to be moved to sys/dev/scsipi only ports@ accepts attachements. please send diffs inline with your message. > > Daniel > > [demime 1.01d removed an attachment of type application/octet-stream which had a name of move_scsi_dir.patch] > sorry, but no. dlg |