Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 1 Dec 2011 12:53:57 -0700 (MST)
From:      Warren Block <wblock@wonkity.com>
To:        Manolis Kiagias <sonic2000gr@gmail.com>
Cc:        freebsd-doc@freebsd.org, bug-followup@freebsd.org
Subject:   Re: docs/162961: [PATCH] Update VirtualBox section in handbook
Message-ID:  <alpine.BSF.2.00.1112011250280.12893@wonkity.com>
In-Reply-To: <4ED7BECE.2060000@gmail.com>
References:  <201111302220.pAUMK8Gt078242@freefall.freebsd.org> <4ED7BECE.2060000@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
  This message is in MIME format.  The first part should be readable text,
  while the remaining parts are likely unreadable without MIME-aware tools.

---902635197-1461172185-1322769237=:12893
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed

On Thu, 1 Dec 2011, Manolis Kiagias wrote:

> Here is the latest version of the patch, with few more minor fixes and a 
> complete whitespace overhaul:
>
> http://people.freebsd.org/~manolis/patches/virtualization-virtualbox-chapter-patch-new.txt
>
> A test build is here:
>
> http://www.freebsdgr.org/all/en_US.ISO8859-1/books/handbook/virtualization-guest.html
>
> If there are no objections / fixes, this goes in tomorrow :)

A few quick suggestions:

--- virtualization-virtualbox-chapter-patch-new.txt.orig	2011-12-01 12:41:46.000000000 -0700
+++ virtualization-virtualbox-chapter-patch-new.txt	2011-12-01 12:52:28.000000000 -0700
@@ -7,7 +7,7 @@
       </sect2>
  +
  +    <sect2 id="virtualization-guest-virtualbox-guest-additions">
-+      <title>&virtualbox; Guest Additions on &os; Guest</title>
++      <title>&virtualbox; Guest Additions on a &os; Guest</title>
  +
  +      <para>The <application>&virtualbox;</application> guest additions
  +	provide support for:</para>
@@ -44,7 +44,7 @@
  +
  +      <screen>&prompt.root; <userinput>cd /usr/ports/emulators/virtualbox-ose-additions && make install clean</userinput></screen>
  +
-+      <para>Add the following lines to <filename>/etc/rc.conf</filename>:</para>
++      <para>Add these lines to <filename>/etc/rc.conf</filename>:</para>
  +
  +      <programlisting>vboxguest_enable="YES"
  +vboxservice_enable="YES"</programlisting>
@@ -55,8 +55,8 @@
  +      <programlisting>vboxservice_flags="--disable-timesync"</programlisting>
  +
  +      <para>The <literal>vboxvideo_drv</literal> should be recognized by
-+	<command>Xorg -configure</command>.  If not, use something like this
-+	in your <filename>xorg.conf</filename> file for the
++	<command>Xorg -configure</command>.  If not, modify
++	<filename>xorg.conf</filename> for the
  +	<application>&virtualbox;</application> video card:</para>
  +
  +      <programlisting>Section "Device"
@@ -71,8 +71,8 @@
  +	BusID "PCI:0:2:0"
  +EndSection</programlisting>
  +
-+      <para>To use <literal>vboxmouse_drv</literal> please adjust the mouse
-+	section in your <filename>xorg.conf</filename> like this:</para>
++      <para>To use <literal>vboxmouse_drv</literal>, adjust the mouse
++	section in <filename>xorg.conf</filename>:</para>
  +
  +      <programlisting>Section "InputDevice"
  +	Identifier "Mouse0"
@@ -127,7 +127,7 @@
  -	may be installed using the following commands:</para>
  +	in <filename role="package">emulators/virtualbox-ose</filename>.
  +	As &virtualbox; is very actively developed, make sure your ports
-+	tree is up to date before installing.  Install using the following
++	tree is up to date before installing.  Install using these
  +	commands:</para>

         <screen>&prompt.root; <userinput>cd /usr/ports/emulators/virtualbox-ose</userinput>
@@ -142,7 +142,7 @@
  -	the functions provided by the &man.sysctl.3; library.</para>
  +      <para>To use the kernel modules that allow bridged or host-only
  +	networking, add the following to <filename>/etc/rc.conf</filename>
-+	and reboot your computer:</para>
++	and reboot the computer:</para>

  -      <para>When using an older version of the port, follow the instructions
  -	below to make sure <filename class="directory">proc</filename> is
@@ -170,28 +170,28 @@
  +      <programlisting>vboxnet_enable="YES"</programlisting>

         <para>The <groupname>vboxusers</groupname> group is created during
- 	the installation of <application>&virtualbox;</application>.
+ 	installation of <application>&virtualbox;</application>.
  @@ -1066,6 +1156,21 @@

         <screen>&prompt.root; <userinput>pw groupmod vboxusers -m <replaceable>yourusername</replaceable></userinput></screen>

  +      <para>The default permissions for <filename
  +	  class="devicefile">/dev/vboxnetctl</filename> are restrictive and
-+	will need to be changed for bridged networking.</para>
++	need to be changed for bridged networking.</para>
  +
  +      <para>To test it temporarily:</para>
  +
  +      <screen>&prompt.root; <userinput>chown root:vboxusers /dev/vboxnetctl</userinput>
  +&prompt.root; <userinput>chmod 0660 /dev/vboxnetctl</userinput></screen>
  +
-+      <para>To make the permissions change permanent, add the following
++      <para>To make the permissions change permanent, add these
  +	lines to <filename>/etc/devfs.conf</filename>:
  +
  +      <programlisting>own     vboxnetctl root:vboxusers
  +perm    vboxnetctl 0660</programlisting>
  +
         <para>To launch <application>&virtualbox;</application>, either select
- 	the <guimenuitem>Sun VirtualBox</guimenuitem> item from your
+ 	the <guimenuitem>Sun VirtualBox</guimenuitem> item from the
   	graphic environment's menu, or type the following in a
  @@ -1081,6 +1186,71 @@
   	visit the relevant page in the &os; wiki, at <ulink
---902635197-1461172185-1322769237=:12893
Content-Type: TEXT/PLAIN; charset=US-ASCII; name=patch-vb-chapter.diff.txt
Content-Transfer-Encoding: BASE64
Content-ID: <alpine.BSF.2.00.1112011253570.12893@wonkity.com>
Content-Description: 
Content-Disposition: attachment; filename=patch-vb-chapter.diff.txt

LS0tIHZpcnR1YWxpemF0aW9uLXZpcnR1YWxib3gtY2hhcHRlci1wYXRjaC1u
ZXcudHh0Lm9yaWcJMjAxMS0xMi0wMSAxMjo0MTo0Ni4wMDAwMDAwMDAgLTA3
MDANCisrKyB2aXJ0dWFsaXphdGlvbi12aXJ0dWFsYm94LWNoYXB0ZXItcGF0
Y2gtbmV3LnR4dAkyMDExLTEyLTAxIDEyOjUyOjI4LjAwMDAwMDAwMCAtMDcw
MA0KQEAgLTcsNyArNyw3IEBADQogICAgICA8L3NlY3QyPg0KICsNCiArICAg
IDxzZWN0MiBpZD0idmlydHVhbGl6YXRpb24tZ3Vlc3QtdmlydHVhbGJveC1n
dWVzdC1hZGRpdGlvbnMiPg0KLSsgICAgICA8dGl0bGU+JnZpcnR1YWxib3g7
IEd1ZXN0IEFkZGl0aW9ucyBvbiAmb3M7IEd1ZXN0PC90aXRsZT4NCisrICAg
ICAgPHRpdGxlPiZ2aXJ0dWFsYm94OyBHdWVzdCBBZGRpdGlvbnMgb24gYSAm
b3M7IEd1ZXN0PC90aXRsZT4NCiArDQogKyAgICAgIDxwYXJhPlRoZSA8YXBw
bGljYXRpb24+JnZpcnR1YWxib3g7PC9hcHBsaWNhdGlvbj4gZ3Vlc3QgYWRk
aXRpb25zDQogKwlwcm92aWRlIHN1cHBvcnQgZm9yOjwvcGFyYT4NCkBAIC00
NCw3ICs0NCw3IEBADQogKw0KICsgICAgICA8c2NyZWVuPiZwcm9tcHQucm9v
dDsgPHVzZXJpbnB1dD5jZCAvdXNyL3BvcnRzL2VtdWxhdG9ycy92aXJ0dWFs
Ym94LW9zZS1hZGRpdGlvbnMgJiYgbWFrZSBpbnN0YWxsIGNsZWFuPC91c2Vy
aW5wdXQ+PC9zY3JlZW4+DQogKw0KLSsgICAgICA8cGFyYT5BZGQgdGhlIGZv
bGxvd2luZyBsaW5lcyB0byA8ZmlsZW5hbWU+L2V0Yy9yYy5jb25mPC9maWxl
bmFtZT46PC9wYXJhPg0KKysgICAgICA8cGFyYT5BZGQgdGhlc2UgbGluZXMg
dG8gPGZpbGVuYW1lPi9ldGMvcmMuY29uZjwvZmlsZW5hbWU+OjwvcGFyYT4N
CiArDQogKyAgICAgIDxwcm9ncmFtbGlzdGluZz52Ym94Z3Vlc3RfZW5hYmxl
PSJZRVMiDQogK3Zib3hzZXJ2aWNlX2VuYWJsZT0iWUVTIjwvcHJvZ3JhbWxp
c3Rpbmc+DQpAQCAtNTUsOCArNTUsOCBAQA0KICsgICAgICA8cHJvZ3JhbWxp
c3Rpbmc+dmJveHNlcnZpY2VfZmxhZ3M9Ii0tZGlzYWJsZS10aW1lc3luYyI8
L3Byb2dyYW1saXN0aW5nPg0KICsNCiArICAgICAgPHBhcmE+VGhlIDxsaXRl
cmFsPnZib3h2aWRlb19kcnY8L2xpdGVyYWw+IHNob3VsZCBiZSByZWNvZ25p
emVkIGJ5DQotKwk8Y29tbWFuZD5Yb3JnIC1jb25maWd1cmU8L2NvbW1hbmQ+
LiAgSWYgbm90LCB1c2Ugc29tZXRoaW5nIGxpa2UgdGhpcw0KLSsJaW4geW91
ciA8ZmlsZW5hbWU+eG9yZy5jb25mPC9maWxlbmFtZT4gZmlsZSBmb3IgdGhl
DQorKwk8Y29tbWFuZD5Yb3JnIC1jb25maWd1cmU8L2NvbW1hbmQ+LiAgSWYg
bm90LCBtb2RpZnkNCisrCTxmaWxlbmFtZT54b3JnLmNvbmY8L2ZpbGVuYW1l
PiBmb3IgdGhlDQogKwk8YXBwbGljYXRpb24+JnZpcnR1YWxib3g7PC9hcHBs
aWNhdGlvbj4gdmlkZW8gY2FyZDo8L3BhcmE+DQogKw0KICsgICAgICA8cHJv
Z3JhbWxpc3Rpbmc+U2VjdGlvbiAiRGV2aWNlIg0KQEAgLTcxLDggKzcxLDgg
QEANCiArCUJ1c0lEICJQQ0k6MDoyOjAiDQogK0VuZFNlY3Rpb248L3Byb2dy
YW1saXN0aW5nPg0KICsNCi0rICAgICAgPHBhcmE+VG8gdXNlIDxsaXRlcmFs
PnZib3htb3VzZV9kcnY8L2xpdGVyYWw+IHBsZWFzZSBhZGp1c3QgdGhlIG1v
dXNlDQotKwlzZWN0aW9uIGluIHlvdXIgPGZpbGVuYW1lPnhvcmcuY29uZjwv
ZmlsZW5hbWU+IGxpa2UgdGhpczo8L3BhcmE+DQorKyAgICAgIDxwYXJhPlRv
IHVzZSA8bGl0ZXJhbD52Ym94bW91c2VfZHJ2PC9saXRlcmFsPiwgYWRqdXN0
IHRoZSBtb3VzZQ0KKysJc2VjdGlvbiBpbiA8ZmlsZW5hbWU+eG9yZy5jb25m
PC9maWxlbmFtZT46PC9wYXJhPg0KICsNCiArICAgICAgPHByb2dyYW1saXN0
aW5nPlNlY3Rpb24gIklucHV0RGV2aWNlIg0KICsJSWRlbnRpZmllciAiTW91
c2UwIg0KQEAgLTEyNyw3ICsxMjcsNyBAQA0KIC0JbWF5IGJlIGluc3RhbGxl
ZCB1c2luZyB0aGUgZm9sbG93aW5nIGNvbW1hbmRzOjwvcGFyYT4NCiArCWlu
IDxmaWxlbmFtZSByb2xlPSJwYWNrYWdlIj5lbXVsYXRvcnMvdmlydHVhbGJv
eC1vc2U8L2ZpbGVuYW1lPi4NCiArCUFzICZ2aXJ0dWFsYm94OyBpcyB2ZXJ5
IGFjdGl2ZWx5IGRldmVsb3BlZCwgbWFrZSBzdXJlIHlvdXIgcG9ydHMNCi0r
CXRyZWUgaXMgdXAgdG8gZGF0ZSBiZWZvcmUgaW5zdGFsbGluZy4gIEluc3Rh
bGwgdXNpbmcgdGhlIGZvbGxvd2luZw0KKysJdHJlZSBpcyB1cCB0byBkYXRl
IGJlZm9yZSBpbnN0YWxsaW5nLiAgSW5zdGFsbCB1c2luZyB0aGVzZQ0KICsJ
Y29tbWFuZHM6PC9wYXJhPg0KICANCiAgICAgICAgPHNjcmVlbj4mcHJvbXB0
LnJvb3Q7IDx1c2VyaW5wdXQ+Y2QgL3Vzci9wb3J0cy9lbXVsYXRvcnMvdmly
dHVhbGJveC1vc2U8L3VzZXJpbnB1dD4NCkBAIC0xNDIsNyArMTQyLDcgQEAN
CiAtCXRoZSBmdW5jdGlvbnMgcHJvdmlkZWQgYnkgdGhlICZtYW4uc3lzY3Rs
LjM7IGxpYnJhcnkuPC9wYXJhPg0KICsgICAgICA8cGFyYT5UbyB1c2UgdGhl
IGtlcm5lbCBtb2R1bGVzIHRoYXQgYWxsb3cgYnJpZGdlZCBvciBob3N0LW9u
bHkNCiArCW5ldHdvcmtpbmcsIGFkZCB0aGUgZm9sbG93aW5nIHRvIDxmaWxl
bmFtZT4vZXRjL3JjLmNvbmY8L2ZpbGVuYW1lPg0KLSsJYW5kIHJlYm9vdCB5
b3VyIGNvbXB1dGVyOjwvcGFyYT4NCisrCWFuZCByZWJvb3QgdGhlIGNvbXB1
dGVyOjwvcGFyYT4NCiAgDQogLSAgICAgIDxwYXJhPldoZW4gdXNpbmcgYW4g
b2xkZXIgdmVyc2lvbiBvZiB0aGUgcG9ydCwgZm9sbG93IHRoZSBpbnN0cnVj
dGlvbnMNCiAtCWJlbG93IHRvIG1ha2Ugc3VyZSA8ZmlsZW5hbWUgY2xhc3M9
ImRpcmVjdG9yeSI+cHJvYzwvZmlsZW5hbWU+IGlzDQpAQCAtMTcwLDI4ICsx
NzAsMjggQEANCiArICAgICAgPHByb2dyYW1saXN0aW5nPnZib3huZXRfZW5h
YmxlPSJZRVMiPC9wcm9ncmFtbGlzdGluZz4NCiAgDQogICAgICAgIDxwYXJh
PlRoZSA8Z3JvdXBuYW1lPnZib3h1c2VyczwvZ3JvdXBuYW1lPiBncm91cCBp
cyBjcmVhdGVkIGR1cmluZw0KLSAJdGhlIGluc3RhbGxhdGlvbiBvZiA8YXBw
bGljYXRpb24+JnZpcnR1YWxib3g7PC9hcHBsaWNhdGlvbj4uDQorIAlpbnN0
YWxsYXRpb24gb2YgPGFwcGxpY2F0aW9uPiZ2aXJ0dWFsYm94OzwvYXBwbGlj
YXRpb24+Lg0KIEBAIC0xMDY2LDYgKzExNTYsMjEgQEANCiAgDQogICAgICAg
IDxzY3JlZW4+JnByb21wdC5yb290OyA8dXNlcmlucHV0PnB3IGdyb3VwbW9k
IHZib3h1c2VycyAtbSA8cmVwbGFjZWFibGU+eW91cnVzZXJuYW1lPC9yZXBs
YWNlYWJsZT48L3VzZXJpbnB1dD48L3NjcmVlbj4NCiAgDQogKyAgICAgIDxw
YXJhPlRoZSBkZWZhdWx0IHBlcm1pc3Npb25zIGZvciA8ZmlsZW5hbWUNCiAr
CSAgY2xhc3M9ImRldmljZWZpbGUiPi9kZXYvdmJveG5ldGN0bDwvZmlsZW5h
bWU+IGFyZSByZXN0cmljdGl2ZSBhbmQNCi0rCXdpbGwgbmVlZCB0byBiZSBj
aGFuZ2VkIGZvciBicmlkZ2VkIG5ldHdvcmtpbmcuPC9wYXJhPg0KKysJbmVl
ZCB0byBiZSBjaGFuZ2VkIGZvciBicmlkZ2VkIG5ldHdvcmtpbmcuPC9wYXJh
Pg0KICsNCiArICAgICAgPHBhcmE+VG8gdGVzdCBpdCB0ZW1wb3JhcmlseTo8
L3BhcmE+DQogKw0KICsgICAgICA8c2NyZWVuPiZwcm9tcHQucm9vdDsgPHVz
ZXJpbnB1dD5jaG93biByb290OnZib3h1c2VycyAvZGV2L3Zib3huZXRjdGw8
L3VzZXJpbnB1dD4NCiArJnByb21wdC5yb290OyA8dXNlcmlucHV0PmNobW9k
IDA2NjAgL2Rldi92Ym94bmV0Y3RsPC91c2VyaW5wdXQ+PC9zY3JlZW4+DQog
Kw0KLSsgICAgICA8cGFyYT5UbyBtYWtlIHRoZSBwZXJtaXNzaW9ucyBjaGFu
Z2UgcGVybWFuZW50LCBhZGQgdGhlIGZvbGxvd2luZw0KKysgICAgICA8cGFy
YT5UbyBtYWtlIHRoZSBwZXJtaXNzaW9ucyBjaGFuZ2UgcGVybWFuZW50LCBh
ZGQgdGhlc2UNCiArCWxpbmVzIHRvIDxmaWxlbmFtZT4vZXRjL2RldmZzLmNv
bmY8L2ZpbGVuYW1lPjoNCiArDQogKyAgICAgIDxwcm9ncmFtbGlzdGluZz5v
d24gICAgIHZib3huZXRjdGwgcm9vdDp2Ym94dXNlcnMNCiArcGVybSAgICB2
Ym94bmV0Y3RsIDA2NjA8L3Byb2dyYW1saXN0aW5nPg0KICsNCiAgICAgICAg
PHBhcmE+VG8gbGF1bmNoIDxhcHBsaWNhdGlvbj4mdmlydHVhbGJveDs8L2Fw
cGxpY2F0aW9uPiwgZWl0aGVyIHNlbGVjdA0KLSAJdGhlIDxndWltZW51aXRl
bT5TdW4gVmlydHVhbEJveDwvZ3VpbWVudWl0ZW0+IGl0ZW0gZnJvbSB5b3Vy
DQorIAl0aGUgPGd1aW1lbnVpdGVtPlN1biBWaXJ0dWFsQm94PC9ndWltZW51
aXRlbT4gaXRlbSBmcm9tIHRoZQ0KICAJZ3JhcGhpYyBlbnZpcm9ubWVudCdz
IG1lbnUsIG9yIHR5cGUgdGhlIGZvbGxvd2luZyBpbiBhDQogQEAgLTEwODEs
NiArMTE4Niw3MSBAQA0KICAJdmlzaXQgdGhlIHJlbGV2YW50IHBhZ2UgaW4g
dGhlICZvczsgd2lraSwgYXQgPHVsaW5rDQo=

---902635197-1461172185-1322769237=:12893--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.00.1112011250280.12893>