Hi,


I’m a student taking part in Google Code-in 2018. The<http://Code-in 2018. The> 
task I am currently working on, 
https://codein.withgoogle.com/dashboard/task-instances/6406170207059968/, 
requires that I review and improve the “Getting Started” tutorial in the 
PostgreSQL docs, and submit a patch to this mailing list.



After reviewing the documentation, I found some parts to be slightly unclear. 
For example, in section 1.3 on creating databases, I found “no response” a bit 
unclear or ambiguous, so I replaced it with “exit without any error messages”.



After some experimentation, I found that a part of the documentation was 
incorrect. In Section 1.3, it was stated that “Database names must have an 
alphabetic first character”. However, I was able to create databases with names 
like “1234”, “$” or even “😀😀”. Hence, I decided to remove that sentence.



A diff of my changes is attached.



Thank you and I would appreciate any feedback that would make my first Postgres 
patch better!

Jun Rong



diff --git a/doc/src/sgml/installation.sgml b/doc/src/sgml/installation.sgml
index 4487d0cfd1..2dc2b82d9a 100644
--- a/doc/src/sgml/installation.sgml
+++ b/doc/src/sgml/installation.sgml
@@ -75,7 +75,7 @@ su - postgres
       <application>make</application> programs or older <acronym>GNU</acronym> 
<application>make</application> versions will <emphasis>not</emphasis> work.
       (<acronym>GNU</acronym> <application>make</application> is sometimes 
installed under
       the name <filename>gmake</filename>.)  To test for <acronym>GNU</acronym>
-      <application>make</application> enter:
+      <application>make</application> and check its version, enter:
 <screen>
 <userinput>make --version</userinput>
 </screen>
@@ -385,8 +385,8 @@ su - postgres
     This script will run a number of tests to determine values for various
     system dependent variables and detect any quirks of your
     operating system, and finally will create several files in the
-    build tree to record what it found.  You can also run
-    <filename>configure</filename> in a directory outside the source
+    build tree to record what it found.  If it does not print any error 
messages, configuration was successful.
+    You can also run <filename>configure</filename> in a directory outside the 
source
     tree, if you want to keep the build directory separate.  This
     procedure is also called a
     <indexterm><primary>VPATH</primary></indexterm><firstterm>VPATH</firstterm>
@@ -1610,6 +1610,15 @@ su - postgres
 <screen>
 All of PostgreSQL successfully made. Ready to install.
 </screen>
+    If you see an error message like:
+<screen>
+ERROR: `flex' is missing on your system. It is needed to create the
+file `bootscanner.c'. You can either get flex from a GNU mirror site
+or download an official distribution of PostgreSQL, which contains
+pre-packaged flex output.
+</screen>
+    then one or more of the packages <productname>PostgreSQL</productname> 
requires is not installed.
+    See <xref linkend="install-requirements"/> for the required packages.
    </para>
 
   <para>
diff --git a/doc/src/sgml/start.sgml b/doc/src/sgml/start.sgml
index 5b73557835..4e3aaaac86 100644
--- a/doc/src/sgml/start.sgml
+++ b/doc/src/sgml/start.sgml
@@ -19,8 +19,8 @@
 
    <para>
     If you are not sure whether <productname>PostgreSQL</productname>
-    is already available or whether you can use it for your
-    experimentation then you can install it yourself.  Doing so is not
+    is already available for your experimentation,
+    you can install it yourself.  Doing so is not
     hard and it can be a good exercise.
     <productname>PostgreSQL</productname> can be installed by any
     unprivileged user; no superuser (<systemitem>root</systemitem>)
@@ -83,7 +83,7 @@
 
      <listitem>
       <para>
-       The user's client (frontend) application that wants to perform
+       The user's client (frontend), an application that wants to perform
        database operations.  Client applications can be very diverse
        in nature:  a client could be a text-oriented tool, a graphical
        application, a web server that accesses the database to
@@ -99,7 +99,7 @@
 
    <para>
     As is typical of client/server applications, the client and the
-    server can be on different hosts.  In that case they communicate
+    server can be on different machines or networks.  In that case they 
communicate
     over a TCP/IP network connection.  You should keep this in mind,
     because the files that can be accessed on a client machine might
     not be accessible (or might only be accessible using a different
@@ -153,7 +153,7 @@
 <screen>
 <prompt>$</prompt> <userinput>createdb mydb</userinput>
 </screen>
-    If this produces no response then this step was successful and you can 
skip over the
+    If this exits without any error message then this step was successful and 
you can skip over the
     remainder of this section.
    </para>
 
@@ -168,7 +168,7 @@ createdb: command not found
 <screen>
 <prompt>$</prompt> <userinput>/usr/local/pgsql/bin/createdb mydb</userinput>
 </screen>
-    The path at your site might be different.  Contact your site
+    The path at your site's server might be different.  Contact your site
     administrator or check the installation instructions to
     correct the situation.
    </para>
@@ -240,12 +240,11 @@ createdb: database creation failed: ERROR:  permission 
denied to create database
    <para>
     You can also create databases with other names.
     <productname>PostgreSQL</productname> allows you to create any
-    number of databases at a given site.  Database names must have an
-    alphabetic first character and are limited to 63 bytes in
-    length.  A convenient choice is to create a database with the same
-    name as your current user name.  Many tools assume that database
-    name as the default, so it can save you some typing.  To create
-    that database, simply type:
+    number of databases at a given site.  Database names are limited to 63 
bytes in
+    length. Database names longer than 63 bytes will be truncated. A 
convenient 
+    choice is to create a database with the same name as your current user 
name. 
+    Many tools assume that database name as the default, so it 
+    can save you some typing. To create that database, simply type:
 <screen>
 <prompt>$</prompt> <userinput>createdb</userinput>
 </screen>
@@ -355,7 +354,7 @@ mydb=#
    <para>
     The last line printed out by <command>psql</command> is the
     prompt, and it indicates that <command>psql</command> is listening
-    to you and that you can type <acronym>SQL</acronym> queries into a
+    to you and that you can type commands and <acronym>SQL</acronym> queries 
into a
     work space maintained by <command>psql</command>.  Try out these
     commands:
     <indexterm><primary>version</primary></indexterm>

Reply via email to