We no longer build PDFs so stop pretending
[privoxy.git] / doc / source / developer-manual.sgml
1 <!DOCTYPE article PUBLIC "-//OASIS//DTD DocBook V3.1//EN"[
2 <!entity % dummy "IGNORE">
3 <!entity supported SYSTEM "supported.sgml">
4 <!entity newfeatures SYSTEM "newfeatures.sgml">
5 <!entity p-intro SYSTEM "privoxy.sgml">
6 <!entity history SYSTEM "history.sgml">
7 <!entity seealso SYSTEM "seealso.sgml">
8 <!entity contacting SYSTEM "contacting.sgml">
9 <!entity copyright SYSTEM "copyright.sgml">
10 <!entity license SYSTEM "license.sgml">
11 <!entity p-version "3.0.20">
12 <!entity p-status "UNRELEASED">
13 <!entity % p-not-stable "INCLUDE">
14 <!entity % p-stable "IGNORE">
15 <!entity % p-text "IGNORE">        <!-- define we are not a text only doc -->
16 <!entity % p-doc "INCLUDE">        <!-- and we are a formal doc           -->
17 <!entity % seealso-extra "INCLUDE"> <!-- extra stuff from seealso.sgml    -->
18 <!entity  my-copy "&copy;">        <!-- kludge for docbook2man            -->
19 ]>
20 <!--
21  File        :  $Source: /cvsroot/ijbswa/current/doc/source/developer-manual.sgml,v $
22
23  Purpose     :  developer manual
24                 This file belongs into
25                 ijbswa.sourceforge.net:/home/groups/i/ij/ijbswa/htdocs/
26
27  $Id: developer-manual.sgml,v 2.43 2012/03/20 13:04:03 fabiankeil Exp $
28
29  Copyright (C) 2001-2012 Privoxy Developers http://www.privoxy.org/
30  See LICENSE.
31
32  ========================================================================
33  NOTE: Please read developer-manual/documentation.html before touching
34  anything in this, or other Privoxy documentation. You have been warned!
35  Failure to abide by this rule will result in the revocation of your license
36  to live a peaceful existence!
37  ========================================================================
38
39 -->
40
41 <article id="index">
42   <artheader>
43     <title>Privoxy Developer Manual</title>
44     <pubdate>
45      <subscript>
46     <!-- Completely the wrong markup, but very little is allowed  -->
47     <!-- in this part of an article. FIXME -->
48       <link linkend="copyright">Copyright</link> &my-copy; 2001-2009 by
49       <ulink url="http://www.privoxy.org/">Privoxy Developers</ulink>
50      </subscript>
51     </pubdate>
52
53
54     <pubdate>$Id: developer-manual.sgml,v 2.43 2012/03/20 13:04:03 fabiankeil Exp $</pubdate>
55
56 <!--
57
58 Note: this should generate a separate page, and a live link to it.
59 But it doesn't for some mysterious reason. Please leave commented
60 unless it can be fixed proper. For the time being, the copyright
61 statement will be in copyright.smgl.
62
63 Hal.
64
65 <legalnotice id="legalnotice">
66  <para>
67   text goes here ........
68  </para>
69 </legalnotice>
70
71 -->
72
73     <abstract>
74
75 <![%dummy;[
76  <para>
77  <comment>
78   This is here to keep vim syntax file from breaking :/
79   If I knew enough to fix it, I would.
80   PLEASE DO NOT REMOVE! HB: hal@foobox.net
81  </comment>
82  </para>
83  ]]>
84 <para>
85  The developer manual provides guidance on coding, testing, packaging, documentation
86  and other issues of importance to those involved with
87  <application>Privoxy</application> development. It is mandatory (and helpful!) reading
88  for anyone who wants to join the team. Note that it's currently out of date
89  and may not be entirely correct. As always, patches are welcome.
90 </para>
91
92 <!-- Include privoxy.sgml boilerplate text: -->
93
94 <!--  &p-intro; Someone interested enough in the project to contribute
95                 will already know at this point what Privoxy is. -->
96
97 <!-- end boilerplate -->
98
99 <para>
100  Please note that this document is constantly evolving. This copy represents
101  the state at the release of version &p-version;.
102  You can find the latest version of the this manual at <ulink
103  url="http://www.privoxy.org/developer-manual/">http://www.privoxy.org/developer-manual/</ulink>.
104  Please see <link linkend="contact">the Contact section</link>
105  on how to contact the developers.
106 </para>
107 <!--        <para> -->
108 <!--    Feel free to send a note to the developers at <email>ijbswa-developers@lists.sourceforge.net</email>. -->
109 <!--   </para> -->
110
111     </abstract>
112   </artheader>
113
114
115 <!--   ~~~~~       New section      ~~~~~     -->
116   <sect1 id="introduction"><title>Introduction</title>
117 <!--
118
119  I don't like seeing blank space :) So added *something* here.
120
121  -->
122     <para>
123      <application>Privoxy</application>, as an heir to
124      <application>Junkbuster</application>, is a Free Software project
125      and the code is licensed under the GNU General Public License version 2.
126      As such, <application>Privoxy</application> development is potentially open
127      to anyone who has the time, knowledge, and desire to contribute
128      in any capacity. Our goals are simply to continue the mission,
129      to improve <application>Privoxy</application>, and
130      to make it available to as wide an audience as possible.
131     </para>
132     <para>
133      One does not have to be a programmer to contribute. Packaging, testing,
134      documenting and porting, are all important jobs as well.
135     </para>
136
137   <!--   ~~~~~       New section      ~~~~~     -->
138   <sect2 id="quickstart"><title>Quickstart to Privoxy Development</title>
139    <para>
140     The first step is to join the <ulink
141       url="mailto:ijbswa-developers@lists.sourceforge.net">developer's mailing list</ulink>.
142     You can submit your ideas, or even better patches. Patches are best
143     submitted to the Sourceforge tracker set up for this purpose, but
144     can be sent to the list for review too.
145    </para>
146     <para>
147      You will also need to have a cvs package installed, which will
148      entail having ssh installed as well (which seems to be a requirement of
149      SourceForge), in order to access the cvs repository. Having the GNU build
150      tools is also going to be important (particularly, autoconf and gmake).
151     </para>
152     <para>
153       For the time being (read, this section is under construction), you can
154       also refer to the extensive comments in the source code. In fact,
155       reading the code is recommended in any case.
156     </para>
157    </sect2>
158   </sect1>
159
160   <!--   ~~~~~       New section      ~~~~~     -->
161   <sect1 id="cvs"><title>The CVS Repository</title>
162     <para>
163       If you become part of the active development team, you will eventually
164       need write access to our holy grail, the CVS repository. One of the
165       team members will need to set this up for you. Please read
166       this chapter completely before accessing via CVS.
167     </para>
168
169     <sect2 id="cvsaccess"><title>Access to CVS</title>
170       <para>
171         The project's CVS repository is hosted on
172         <ulink url="http://sourceforge.net/">SourceForge.</ulink>
173         Please refer to the chapters 6 and 7 in
174         <ulink url="http://sourceforge.net/docman/?group_id=1">SF's site
175         documentation</ulink> for the technical access details for your
176         operating system. For historical reasons, the CVS server is
177         called <literal>ijbswa.cvs.sourceforge.net</literal>, the repository is
178         called <literal>ijbswa</literal>, and the source tree module is called
179         <literal>current</literal>.
180       </para>
181     </sect2>
182
183     <sect2 id="cvsbranches">
184     <title>Branches</title>
185      <para>
186        Within the CVS repository, there are modules and branches. As
187        mentioned, the sources are in the <literal>current</literal>
188        <quote>module</quote>. Other modules are present for platform specific
189        issues. There is a webview of the CVS hierarchy at <ulink
190        url="http://ijbswa.cvs.sourceforge.net/ijbswa/">http://ijbswa.cvs.sourceforge.net/ijbswa/</ulink>,
191        which might help with visualizing how these pieces fit together.
192      </para>
193      <!--
194      <para>
195        Branches are used to fork a sub-development path from the main trunk.
196        Within the <literal>current</literal> module where the sources are, there
197        is always at least one <quote>branch</quote> from the main trunk
198        devoted to a stable release series. The main trunk is where active
199        development takes place for the next stable series (e.g. 3.2.x).
200        So just prior to each stable series (e.g. 3.0.x), a branch is created
201        just for stable series releases (e.g. 3.0.0 -> 3.0.1 -> 3.0.2, etc).
202        Once the initial stable release of any stable branch has taken place,
203        this branch is <emphasis>only used for bugfixes</emphasis>, which have
204        had prior testing before being committed to CVS. (See <link
205        linkend="versionnumbers">Version Numbers</link> below for details on
206        versioning.)
207      </para>
208      -->
209      <para>
210       At one time there were two distinct branches: stable and unstable. The
211       more drastic changes were to be in the unstable branch. These branches
212       have now been merged to minimize time and effort of maintaining two
213       branches.
214      </para>
215     <!--
216      <para>
217        This will result in at least two active branches, which means there may
218        be occasions that require the same (or similar) item to be
219        checked into to two different places (assuming its a bugfix and needs
220        fixing in both the stable and unstable trees). This also means that in
221        order to have access to both trees, both will have to be checked out
222        separately. Use the <literal>cvs -r</literal> flag to check out a
223        branch, e.g: <literal>cvs co -r v_3_0_branch current</literal>.
224      </para>
225     -->
226     </sect2>
227
228     <sect2 id="cvscommit"><title>CVS Commit Guidelines</title>
229       <para>
230         The source tree is the heart of every software project. Every effort must
231         be made to ensure that it is readable, compilable and consistent at all
232         times. <!-- There are differing guidelines for the stable branch and the
233         main development trunk, and --> We expect anyone with CVS access to strictly
234         adhere to the following guidelines:
235       </para>
236
237       <para>
238        Basic Guidelines, for all branches:
239       </para>
240       <para>
241         <itemizedlist>
242           <listitem><para>
243             Please don't commit even
244             a small change without testing it thoroughly first. When we're
245             close to a public release, ask a fellow developer to review your
246             changes.
247           </para></listitem>
248           <listitem><para>
249             Your commit message should give a concise overview of <emphasis>what you
250             changed</emphasis> (no big details) and <emphasis>why you changed it</emphasis>
251             Just check previous messages for good examples.
252           </para></listitem>
253           <listitem><para>
254             Don't use the same message on multiple files, unless it equally applies to
255             all those files.
256           </para></listitem>
257           <listitem><para>
258             If your changes span multiple files, and the code won't recompile unless
259             all changes are committed (e.g. when changing the signature of a function),
260             then commit all files one after another, without long delays in between.
261             If necessary, prepare the commit messages in advance.
262           </para></listitem>
263           <listitem><para>
264             Before changing things on CVS, make sure that your changes are in line
265             with the team's general consensus on what should be done.
266           </para></listitem>
267           <listitem>
268            <para>
269             Note that near a major public release, we get more cautious.
270             There is always the possibility to submit a patch to the <ulink
271             url="http://sourceforge.net/tracker/?atid=311118&amp;group_id=11118&amp;func=browse">patch
272             tracker</ulink> instead.
273           </para>
274          </listitem>
275         </itemizedlist>
276       </para>
277
278 <!--
279       <para>
280        Stable branches are handled with more care, especially after the
281        initial *.*.0 release, and we are just in bugfix mode. In addition to
282        the above, the below applies only to the stable branch (currently the
283        <literal>v_3_0_branch</literal> branch):
284       </para>
285
286       <para>
287        <itemizedlist>
288         <listitem>
289          <para>
290            Do not commit <emphasis>anything</emphasis> unless your proposed
291            changes have been well tested first, preferably by other members of the
292            project, or have prior approval of the project leaders or consensus
293            of the devel list.
294          </para>
295         </listitem>
296        <listitem>
297         <para>
298          Where possible, bugfixes and changes should be tested in the main
299          development trunk first. There may be occasions where this is not
300          feasible, though.
301         </para>
302        </listitem>
303        <listitem>
304         <para>
305           Alternately, proposed changes can be submitted as patches to the patch tracker on
306           Sourceforge first: <ulink
307           url="http://sourceforge.net/tracker/?group_id=11118&#38;atid=311118">http://sourceforge.net/tracker/?group_id=11118&#38;atid=311118</ulink>.
308           Then ask for peer review.
309         </para>
310        </listitem>
311         <listitem>
312          <para>
313           Do not even think about anything except bugfixes. No new features!
314          </para>
315         </listitem>
316
317        </itemizedlist>
318       </para>
319     -->
320     </sect2>
321
322   </sect1>
323
324   <!--   ~~~~~       New section      ~~~~~     -->
325 <sect1 id="documentation"><title>Documentation Guidelines</title>
326   <para>
327     All formal documents are maintained in Docbook SGML and located in the
328     <computeroutput>doc/source/*</computeroutput> directory. You will need
329     <ulink url="http://www.docbook.org">Docbook</ulink>, the Docbook
330     DTD's and the Docbook modular stylesheets (or comparable alternatives),
331     and either <application>jade</application> or
332     <application>openjade</application> (recommended) installed in order to
333     build docs from source. Currently there is <ulink
334     url="../user-manual/index.html"><citetitle>user-manual</citetitle></ulink>,
335     <ulink url="../faq/index.html"><citetitle>FAQ</citetitle></ulink>, and, of
336     course this, the <citetitle>developer-manual</citetitle> in this format.
337     The <citetitle>README</citetitle>, <citetitle>AUTHORS</citetitle>,
338     <citetitle>INSTALL</citetitle>,
339     <citetitle>privoxy.1</citetitle> (man page), and
340     <citetitle>config</citetitle> files are also now maintained as Docbook
341     SGML. These files, when built, in the top-level source directory are
342     generated files! Also, the <application>Privoxy</application> <filename>index.html</filename> (and a
343     variation on this file, <filename>privoxy-index.html</filename>,
344     meant for inclusion with doc packages), are maintained as SGML as well.
345     <emphasis>DO NOT edit these directly</emphasis>. Edit the SGML source, or
346     contact someone involved in the documentation.
347     </para>
348     <para>
349      <filename>config</filename> requires some special handling. The reason it
350      is maintained this way is so that the extensive comments in the file
351      mirror those in <citetitle>user-manual</citetitle>. But the conversion
352      process requires going from SGML to HTML to text to special formatting
353      required for the embedded comments. Some of this does not survive so
354      well. Especially some of the examples that are longer than 80 characters.
355      The build process for this file outputs to <filename>config.new</filename>,
356      which should be reviewed for errors and mis-formatting. Once satisfied
357      that it is correct, then it should be hand copied to
358      <filename>config</filename>.
359     </para>
360     <para>
361      Other, less formal documents (e.g. <filename>LICENSE</filename>) are
362      maintained as plain text files in the top-level source directory.
363     </para>
364     <para>
365      Packagers are encouraged to include this documentation. For those without
366      the ability to build the docs locally, text versions of each are kept in
367      CVS. HTML versions are also being kept in CVS under
368      <filename>doc/webserver/*</filename>.
369     </para>
370     <para>
371      Formal documents are built with the Makefile targets of
372      <computeroutput>make dok</computeroutput>, or alternately
373      <computeroutput>make redhat-dok</computeroutput>. If you have problems,
374      try both. The build process uses the document SGML sources in
375      <computeroutput>doc/source/*/*</computeroutput> to update all text files in
376      <computeroutput>doc/text/</computeroutput> and to update all HTML
377      documents in <computeroutput>doc/webserver/</computeroutput>.
378     </para>
379     <para>
380      Documentation writers should please make sure documents build
381      successfully before committing to CVS, if possible.
382     </para>
383     <para>
384      How do you update the webserver (i.e. the pages on privoxy.org)?
385
386      <orderedlist numeration="arabic">
387       <listitem><para>
388         First, build the docs by running <computeroutput>make
389         dok</computeroutput> (or alternately <computeroutput>make
390         redhat-dok</computeroutput>). For PDF docs, do <computeroutput>make
391         dok-pdf</computeroutput>.
392       </para></listitem>
393       <listitem><para>
394         Run <computeroutput>make webserver</computeroutput> which copies all
395         files from <computeroutput>doc/webserver</computeroutput> to the
396         sourceforge webserver via scp.
397       </para></listitem>
398      </orderedlist>
399   </para>
400
401   <para>
402    Finished docs should be occasionally submitted to CVS
403    (<filename>doc/webserver/*/*.html</filename>) so that those without
404    the ability to build them locally, have access to them if needed.
405    This is especially important just prior to a new release! Please
406    do this <emphasis>after</emphasis> the <literal>$VERSION</literal> and
407    other release specific data in <filename>configure.in</filename> has been
408    updated (this is done just prior to a new release).
409   </para>
410
411 <!--   ~~~~~       New section      ~~~~~     -->
412 <sect2 id="sgml">
413 <title>Quickstart to Docbook and SGML</title>
414 <para>
415  If you are not familiar with SGML, it is a markup language similar to HTML.
416  Actually, not a mark up language per se, but a language used to define
417  markup languages. In fact, HTML is an SGML application. Both will use
418  <quote>tags</quote> to format text and other content. SGML tags can be much
419  more varied, and flexible, but do much of the same kinds of things. The tags,
420  or <quote>elements</quote>, are definable in SGML. There is no set
421  <quote>standards</quote>. Since we are using
422  <application>Docbook</application>, our tags are those that are defined by
423  <application>Docbook</application>. Much of how the finish document is
424  rendered is determined by the <quote>stylesheets</quote>.
425  The stylesheets determine how each tag gets translated to HTML, or other
426  formats.
427 </para>
428
429 <para>
430  Tags in Docbook SGML need to be always <quote>closed</quote>. If not, you
431  will likely generate errors. Example: <literal>&lt;title&gt;My
432  Title&lt;/title&gt;</literal>. They are also case-insensitive, but we
433  strongly suggest using all lower case. This keeps compatibility with
434  [Docbook] <application>XML</application>.
435 </para>
436
437 <para>
438  Our documents use <quote>sections</quote> for the most part. Sections
439  will be processed into HTML headers (e.g. <literal>h1</literal> for
440  <literal>sect1</literal>). The <application>Docbook</application> stylesheets
441  will use these to also generate the Table of Contents for each doc. Our
442  TOC's are set to a depth of three. Meaning <literal>sect1</literal>,
443  <literal>sect2</literal>, and <literal>sect3</literal> will have TOC
444  entries, but <literal>sect4</literal> will not. Each section requires
445  a <literal>&lt;title&gt;</literal> element, and at least one
446  <literal>&lt;para&gt;</literal>. There is a limit of five section
447  levels in Docbook, but generally three should be sufficient for our
448  purposes.
449 </para>
450
451 <para>
452  Some common elements that you likely will use:
453 </para>
454
455 <para>
456   <simplelist>
457     <member>
458       <emphasis>&lt;para&gt;&lt;/para&gt;</emphasis>, paragraph delimiter. Most
459       text needs to be within paragraph elements (there are some exceptions).
460     </member>
461     <member>
462       <emphasis>&lt;emphasis&gt;&lt;/emphasis&gt;</emphasis>, the stylesheets
463       make this italics.
464     </member>
465     <member>
466       <emphasis>&lt;filename&gt;&lt;/filename&gt;</emphasis>, files and directories.
467     </member>
468     <member>
469       <emphasis>&lt;command&gt;&lt;/command&gt;</emphasis>, command examples.
470     </member>
471     <member>
472       <emphasis>&lt;literallayout&gt;&lt;/literallayout&gt;</emphasis>, like
473       <literal>&lt;pre&gt;</literal>, more or less.
474     </member>
475     <member>
476       <emphasis>&lt;itemizedlist&gt;&lt;/itemizedlist&gt;</emphasis>, list with bullets.
477     </member>
478     <member>
479       <emphasis>&lt;listitem&gt;&lt;/listitem&gt;</emphasis>, member of the above.
480     </member>
481     <member>
482       <emphasis>&lt;screen&gt;&lt;/screen&gt;</emphasis>, screen output, implies
483       <literal>&lt;literallayout&gt;</literal>.
484     </member>
485     <member>
486       <emphasis>&lt;ulink url="example.com"&gt;&lt;/ulink&gt;</emphasis>, like
487       HTML <literal>&lt;a&gt;</literal> tag.
488     </member>
489     <member>
490       <emphasis>&lt;quote&gt;&lt;/quote&gt;</emphasis>, for, doh, quoting text.
491     </member>
492   </simplelist>
493 </para>
494
495 <para>
496  Look at any of the existing docs for examples of all these and more.
497 </para>
498
499 <para>
500  You might also find <quote><ulink
501  url="http://opensource.bureau-cornavin.com/crash-course/index.html">Writing Documentation
502  Using DocBook - A Crash Course</ulink></quote> useful.
503 </para>
504 </sect2>
505
506 <!--   ~~~~~       New section      ~~~~~     -->
507   <sect2 id="docstyle">
508   <title><application>Privoxy</application> Documentation Style</title>
509    <para>
510     It will be easier if everyone follows a similar writing style. This
511     just makes it easier to read what someone else has written if it
512     is all done in a similar fashion.
513    </para>
514    <para>
515     Here it is:
516    </para>
517    <para>
518     <itemizedlist>
519      <listitem>
520       <para>
521        All tags should be lower case.
522       </para>
523     </listitem>
524     <listitem>
525      <para>
526        Tags delimiting a <emphasis>block</emphasis> of text (even small
527        blocks) should be on their own line. Like:
528        <literallayout>
529  &lt;para&gt;
530   Some text goes here.
531  &lt;/para&gt;
532        </literallayout>
533        Tags marking individual words, or few words, should be in-line:
534        <literallayout>
535   Just to &lt;emphasis&gt;emphasize&lt;/emphasis&gt;, some text goes here.
536        </literallayout>
537      </para>
538    </listitem>
539    <listitem>
540     <para>
541       Tags should be nested and step indented for block text like: (except
542       in-line tags)
543      <literallayout>
544  &lt;para&gt;
545   &lt;itemizedlist&gt;
546    &lt;para&gt;
547     &lt;listitem&gt;
548       Some text goes here in our list example.
549      &lt;/listitem&gt;
550    &lt;/para&gt;
551   &lt;/itemizedlist&gt;
552  &lt;/para&gt;
553        </literallayout>
554       This makes it easier to find the text amongst the tags ;-)
555     </para>
556    </listitem>
557    <listitem>
558     <para>
559      Use white space to separate logical divisions within a document,
560      like between sections. Running everything together consistently
561      makes it harder to read and work on.
562     </para>
563    </listitem>
564    <listitem>
565     <para>
566      Do not hesitate to make comments. Comments can either use the
567      &lt;comment&gt; element, or the &lt;!--  --&gt; style comment
568      familiar from HTML. (Note in Docbook v4.x &lt;comment&gt; is
569      replaced by &lt;remark&gt;.)
570     </para>
571   </listitem>
572   <listitem>
573    <para>
574      We have an international audience. Refrain from slang, or English
575      idiosyncrasies (too many to list :). Humor also does not translate
576      well sometimes.
577    </para>
578   </listitem>
579   <listitem>
580    <para>
581     Try to keep overall line lengths in source files to 80 characters or less
582     for obvious reasons. This is not always possible, with lengthy URLs for
583     instance.
584    </para>
585   </listitem>
586   <listitem>
587    <para>
588     Our documents are available in differing formats. Right now, they
589     are just plain text and/or HTML, but others are always a
590     future possibility. Be careful with URLs (&lt;ulink&gt;), and avoid
591     this mistake:
592    </para>
593    <para>
594      My favorite site is &lt;ulink url="http://example.com"&gt;here&lt;/ulink&gt;.
595    </para>
596    <para>
597      This will render as <quote>My favorite site is here</quote>, which is
598      not real helpful in a text doc. Better like this:
599    </para>
600    <para>
601      My favorite site is &lt;ulink url="http://example.com"&gt;example.com&lt;/ulink&gt;.
602    </para>
603   </listitem>
604   <listitem>
605    <para>
606     All documents should be spell checked occasionally.
607     <application>aspell</application> can check SGML with the
608     <literal>-H</literal> option. (<application>ispell</application> I think
609     too.)
610    </para>
611   </listitem>
612
613   </itemizedlist>
614  </para>
615
616   </sect2>
617
618
619  <!--   ~~~~~       New section      ~~~~~     -->
620
621  <sect2><title>Privoxy Custom Entities</title>
622  <para>
623   <application>Privoxy</application> documentation is using
624   a number of customized <quote>entities</quote> to facilitate
625   documentation maintenance.
626  </para>
627  <para>
628   We are using a set of <quote>boilerplate</quote> files with generic text,
629   that is used by multiple docs. This way we can write something once, and use
630   it repeatedly without having to re-write the same content over and over again.
631   If editing such a file, keep in mind that it should be
632   <emphasis>generic</emphasis>. That is the purpose; so it can be used in varying
633   contexts without additional modifications.
634  </para>
635  <para>
636   We are also using what <application>Docbook</application> calls
637   <quote>internal entities</quote>. These are like variables in
638   programming. Well, sort of. For instance, we have the
639   <literal>p-version</literal> entity that contains the current
640   <application>Privoxy</application> version string. You are strongly
641   encouraged to use these where possible. Some of these obviously
642   require re-setting with each release (done by the Makefile). A sampling of
643   custom entities are listed below. See any of the main docs for examples.
644  </para>
645
646  <para>
647   <itemizedlist>
648   <listitem>
649    <para>
650     Re- <quote>boilerplate</quote> text entities are defined like:
651    </para>
652    <para>
653     <literal>&lt;!entity supported SYSTEM "supported.sgml"&gt;</literal>
654    </para>
655    <para>
656      In this example, the contents of the file,
657      <filename>supported.sgml</filename> is available for inclusion anywhere
658      in the doc. To make this happen, just reference the now defined
659      entity: <literal>&#38;supported;</literal> (starts with an ampersand
660      and ends with a semi-colon), and the contents will be dumped into
661      the finished doc at that point.
662    </para>
663   </listitem>
664   <listitem>
665    <para>
666     Commonly used <quote>internal entities</quote>:
667   </para>
668   <simplelist>
669    <member>
670     <emphasis>p-version</emphasis>: the <application>Privoxy</application>
671     version string, e.g. <quote>&p-version;</quote>.
672    </member>
673    <member>
674     <emphasis>p-status</emphasis>: the project status, either
675     <quote>alpha</quote>, <quote>beta</quote>, or <quote>stable</quote>.
676    </member>
677    <member>
678     <emphasis>p-not-stable</emphasis>: use to conditionally include
679     text in <quote>not stable</quote> releases (e.g. <quote>beta</quote>).
680    </member>
681    <member>
682     <emphasis>p-stable</emphasis>: just the opposite.
683    </member>
684    <member>
685     <emphasis>p-text</emphasis>: this doc is only generated as text.
686    </member>
687   </simplelist>
688  </listitem>
689  </itemizedlist>
690  </para>
691  <para>
692   There are others in various places that are defined for a specific
693   purpose. Read the source!
694  </para>
695
696  </sect2>
697
698  </sect1>
699
700 <!--     <listitem><para>be consistent with the redirect script (i.e. the <application>Privoxy</application> program -->
701 <!--       points via the redirect URL at sf to valid end-points in the document)</para></listitem> -->
702
703   <!--   ~~~~~       New section      ~~~~~     -->
704   <sect1 id="coding"><title>Coding Guidelines</title>
705
706     <sect2 id="s1"><title>Introduction</title>
707
708     <para>This set of standards is designed to make our lives easier.  It is
709     developed with the simple goal of helping us keep the "new and improved
710     <application>Privoxy</application>" consistent and reliable. Thus making
711     maintenance easier and increasing chances of success of the
712     project.</para>
713
714     <para>And that of course comes back to us as individuals. If we can
715     increase our development and product efficiencies then we can solve more
716     of the request for changes/improvements and in general feel good about
717     ourselves. ;-></para>
718
719   </sect2>
720
721     <sect2 id="s2"><title>Using Comments</title>
722
723
724     <sect3 id="s3"><title>Comment, Comment, Comment</title>
725
726     <para><emphasis>Explanation:</emphasis></para>
727
728     <para>Comment as much as possible without commenting the obvious.
729     For example do not comment "variable_a is equal to variable_b".
730     Instead explain why variable_a should be equal to the variable_b.
731     Just because a person can read code does not mean they will
732     understand why or what is being done. A reader may spend a lot
733     more time figuring out what is going on when a simple comment
734     or explanation would have prevented the extra research. Please
735     help your brother IJB'ers out!</para>
736
737     <para>The comments will also help justify the intent of the code.
738     If the comment describes something different than what the code
739     is doing then maybe a programming error is occurring.</para>
740
741     <para><emphasis>Example:</emphasis></para>
742 <programlisting>
743 /* if page size greater than 1k ... */
744 if ( page_length() > 1024 )
745 {
746     ... "block" the page up ...
747 }
748
749 /* if page size is small, send it in blocks */
750 if ( page_length() > 1024 )
751 {
752     ... "block" the page up ...
753 }
754
755 This demonstrates 2 cases of "what not to do".  The first is a
756 "syntax comment".  The second is a comment that does not fit what
757 is actually being done.
758 </programlisting>
759   </sect3>
760
761
762
763     <sect3 id="s4"><title>Use blocks for comments</title>
764
765     <para><emphasis>Explanation:</emphasis></para>
766
767     <para>Comments can help or they can clutter. They help when they
768     are differentiated from the code they describe. One line
769     comments do not offer effective separation between the comment
770     and the code. Block identifiers do, by surrounding the code
771     with a clear, definable pattern.</para>
772
773     <para><emphasis>Example:</emphasis></para>
774 <programlisting>
775 /*********************************************************************
776  * This will stand out clearly in your code!
777  *********************************************************************/
778 if ( this_variable == that_variable )
779 {
780    do_something_very_important();
781 }
782
783
784 /* unfortunately, this may not */
785 if ( this_variable == that_variable )
786 {
787    do_something_very_important();
788 }
789
790
791 if ( this_variable == that_variable ) /* this may not either */
792 {
793    do_something_very_important();
794 }</programlisting>
795
796     <para><emphasis>Exception:</emphasis></para>
797
798     <para>If you are trying to add a small logic comment and do not
799     wish to "disrupt" the flow of the code, feel free to use a 1
800     line comment which is NOT on the same line as the code.</para>
801
802
803   </sect3>
804
805
806     <sect3 id="s5"><title>Keep Comments on their own line</title>
807
808     <para><emphasis>Explanation:</emphasis></para>
809
810     <para>It goes back to the question of readability. If the comment
811     is on the same line as the code it will be harder to read than
812     the comment that is on its own line.</para>
813
814     <para>There are three exceptions to this rule, which should be
815     violated freely and often: during the definition of variables,
816     at the end of closing braces, when used to comment
817     parameters.</para>
818
819     <para><emphasis>Example:</emphasis></para>
820 <programlisting>
821 /*********************************************************************
822  * This will stand out clearly in your code,
823  * But the second example won't.
824  *********************************************************************/
825 if ( this_variable == this_variable )
826 {
827    do_something_very_important();
828 }
829
830 if ( this_variable == this_variable ) /*can you see me?*/
831 {
832    do_something_very_important(); /*not easily*/
833 }
834
835
836 /*********************************************************************
837  * But, the encouraged exceptions:
838  *********************************************************************/
839 int urls_read     = 0;     /* # of urls read + rejected */
840 int urls_rejected = 0;     /* # of urls rejected */
841
842 if ( 1 == X )
843 {
844    do_something_very_important();
845 }
846
847
848 short do_something_very_important(
849    short firstparam,   /* represents something */
850    short nextparam     /* represents something else */ )
851 {
852    ...code here...
853
854 }   /* -END- do_something_very_important */
855 </programlisting>
856   </sect3>
857
858
859     <sect3 id="s6"><title>Comment each logical step</title>
860
861     <para><emphasis>Explanation:</emphasis></para>
862
863     <para>Logical steps should be commented to help others follow the
864     intent of the written code and comments will make the code more
865     readable.</para>
866
867     <para>If you have 25 lines of code without a comment, you should
868     probably go back into it to see where you forgot to put
869     one.</para>
870
871     <para>Most "for", "while", "do", etc... loops _probably_ need a
872     comment. After all, these are usually major logic
873     containers.</para>
874
875
876   </sect3>
877
878
879     <sect3 id="s7"><title>Comment All Functions Thoroughly</title>
880
881     <para><emphasis>Explanation:</emphasis></para>
882
883     <para>A reader of the code should be able to look at the comments
884     just prior to the beginning of a function and discern the
885     reason for its existence and the consequences of using it. The
886     reader should not have to read through the code to determine if
887     a given function is safe for a desired use. The proper
888     information thoroughly presented at the introduction of a
889     function not only saves time for subsequent maintenance or
890     debugging, it more importantly aids in code reuse by allowing a
891     user to determine the safety and applicability of any function
892     for the problem at hand. As a result of such benefits, all
893     functions should contain the information presented in the
894     addendum section of this document.</para>
895
896
897   </sect3>
898
899
900     <sect3 id="s8"><title>Comment at the end of braces if the
901     content is more than one screen length</title>
902
903     <para><emphasis>Explanation:</emphasis></para>
904
905     <para>Each closing brace should be followed on the same line by a
906     comment that describes the origination of the brace if the
907     original brace is off of the screen, or otherwise far away from
908     the closing brace. This will simplify the debugging,
909     maintenance, and readability of the code.</para>
910
911     <para>As a suggestion , use the following flags to make the
912     comment and its brace more readable:</para>
913
914     <para>use following a closing brace: } /* -END- if() or while ()
915     or etc... */</para>
916
917     <para><emphasis>Example:</emphasis></para>
918 <programlisting>
919 if ( 1 == X )
920 {
921    do_something_very_important();
922    ...some long list of commands...
923 } /* -END- if x is 1 */
924
925 or:
926
927 if ( 1 == X )
928 {
929    do_something_very_important();
930    ...some long list of commands...
931 } /* -END- if ( 1 == X ) */
932 </programlisting>
933   </sect3>
934
935   </sect2>
936
937     <sect2 id="s9"><title>Naming Conventions</title>
938
939
940
941     <sect3 id="s10"><title>Variable Names</title>
942
943     <para><emphasis>Explanation:</emphasis></para>
944
945     <para>Use all lowercase, and separate words via an underscore
946     ('_'). Do not start an identifier with an underscore. (ANSI C
947     reserves these for use by the compiler and system headers.) Do
948     not use identifiers which are reserved in ANSI C++. (E.g.
949     template, class, true, false, ...). This is in case we ever
950     decide to port Privoxy to C++.</para>
951
952     <para><emphasis>Example:</emphasis></para>
953 <programlisting>
954 int ms_iis5_hack = 0;</programlisting>
955
956     <para><emphasis>Instead of:</emphasis></para>
957
958     <para>
959 <programlisting>
960 int msiis5hack = 0; int msIis5Hack = 0;
961 </programlisting>
962 </para>
963
964
965
966   </sect3>
967
968     <sect3 id="s11"><title>Function Names</title>
969
970     <para><emphasis>Explanation:</emphasis></para>
971
972     <para>Use all lowercase, and separate words via an underscore
973     ('_'). Do not start an identifier with an underscore. (ANSI C
974     reserves these for use by the compiler and system headers.) Do
975     not use identifiers which are reserved in ANSI C++. (E.g.
976     template, class, true, false, ...). This is in case we ever
977     decide to port Privoxy to C++.</para>
978
979     <para><emphasis>Example:</emphasis></para>
980 <programlisting>
981 int load_some_file( struct client_state *csp )</programlisting>
982
983     <para><emphasis>Instead of:</emphasis></para>
984
985     <para>
986 <programlisting>
987 int loadsomefile( struct client_state *csp )
988 int loadSomeFile( struct client_state *csp )
989 </programlisting>
990 </para>
991
992
993   </sect3>
994
995
996     <sect3 id="s12"><title>Header file prototypes</title>
997
998     <para><emphasis>Explanation:</emphasis></para>
999
1000     <para>Use a descriptive parameter name in the function prototype
1001     in header files. Use the same parameter name in the header file
1002     that you use in the c file.</para>
1003
1004     <para><emphasis>Example:</emphasis></para>
1005 <programlisting>
1006 (.h) extern int load_aclfile( struct client_state *csp );
1007 (.c) int load_aclfile( struct client_state *csp )</programlisting>
1008
1009     <para><emphasis>Instead of:</emphasis>
1010 <programlisting>
1011 (.h) extern int load_aclfile( struct client_state * ); or
1012 (.h) extern int load_aclfile();
1013 (.c) int load_aclfile( struct client_state *csp )
1014 </programlisting>
1015 </para>
1016
1017
1018   </sect3>
1019
1020
1021     <sect3 id="s13"><title>Enumerations, and #defines</title>
1022
1023     <para><emphasis>Explanation:</emphasis></para>
1024
1025     <para>Use all capital letters, with underscores between words. Do
1026     not start an identifier with an underscore. (ANSI C reserves
1027     these for use by the compiler and system headers.)</para>
1028
1029     <para><emphasis>Example:</emphasis></para>
1030 <programlisting>
1031 (enumeration) : enum Boolean { FALSE, TRUE };
1032 (#define) : #define DEFAULT_SIZE 100;</programlisting>
1033
1034     <para><emphasis>Note:</emphasis> We have a standard naming scheme for #defines
1035     that toggle a feature in the preprocessor: FEATURE_>, where
1036     > is a short (preferably 1 or 2 word) description.</para>
1037
1038     <para><emphasis>Example:</emphasis></para>
1039 <programlisting>
1040 #define FEATURE_FORCE 1
1041
1042 #ifdef FEATURE_FORCE
1043 #define FORCE_PREFIX blah
1044 #endif /* def FEATURE_FORCE */
1045 </programlisting>
1046   </sect3>
1047
1048
1049     <sect3 id="s14"><title>Constants</title>
1050
1051     <para><emphasis>Explanation:</emphasis></para>
1052
1053     <para>Spell common words out entirely (do not remove vowels).</para>
1054
1055     <para>Use only widely-known domain acronyms and abbreviations.
1056     Capitalize all letters of an acronym.</para>
1057
1058     <para>Use underscore (_) to separate adjacent acronyms and
1059     abbreviations. Never terminate a name with an underscore.</para>
1060
1061     <para><emphasis>Example:</emphasis></para>
1062 <programlisting>
1063 #define USE_IMAGE_LIST 1</programlisting>
1064
1065     <para><emphasis>Instead of:</emphasis></para>
1066
1067     <para>
1068 <programlisting>
1069 #define USE_IMG_LST 1 or
1070 #define _USE_IMAGE_LIST 1 or
1071 #define USE_IMAGE_LIST_ 1 or
1072 #define use_image_list 1 or
1073 #define UseImageList 1
1074 </programlisting>
1075 </para>
1076
1077
1078   </sect3>
1079
1080   </sect2>
1081
1082
1083     <sect2 id="s15"><title>Using Space</title>
1084
1085
1086
1087     <sect3 id="s16"><title>Put braces on a line by themselves.</title>
1088
1089     <para><emphasis>Explanation:</emphasis></para>
1090
1091     <para>The brace needs to be on a line all by itself, not at the
1092     end of the statement. Curly braces should line up with the
1093     construct that they're associated with. This practice makes it
1094     easier to identify the opening and closing braces for a
1095     block.</para>
1096
1097     <para><emphasis>Example:</emphasis></para>
1098 <programlisting>
1099 if ( this == that )
1100 {
1101    ...
1102 }</programlisting>
1103
1104     <para><emphasis>Instead of:</emphasis></para>
1105
1106     <para>if ( this == that ) { ... }</para>
1107
1108     <para>or</para>
1109
1110     <para>if ( this == that ) { ... }</para>
1111
1112     <para><emphasis>Note:</emphasis> In the special case that the if-statement is
1113     inside a loop, and it is trivial, i.e. it tests for a
1114     condition that is obvious from the purpose of the block,
1115     one-liners as above may optically preserve the loop structure
1116     and make it easier to read.</para>
1117
1118     <para><emphasis>Status:</emphasis> developer-discretion.</para>
1119
1120     <para><emphasis>Example exception:</emphasis></para>
1121 <programlisting>
1122 while ( more lines are read )
1123 {
1124    /* Please document what is/is not a comment line here */
1125    if ( it's a comment ) continue;
1126
1127    do_something( line );
1128 }
1129 </programlisting>
1130   </sect3>
1131
1132
1133     <sect3 id="s17"><title>ALL control statements should have a
1134     block</title>
1135
1136     <para><emphasis>Explanation:</emphasis></para>
1137
1138     <para>Using braces to make a block will make your code more
1139     readable and less prone to error. All control statements should
1140     have a block defined.</para>
1141
1142     <para><emphasis>Example:</emphasis></para>
1143 <programlisting>
1144 if ( this == that )
1145 {
1146    do_something();
1147    do_something_else();
1148 }</programlisting>
1149
1150     <para><emphasis>Instead of:</emphasis></para>
1151
1152     <para>if ( this == that ) do_something(); do_something_else();</para>
1153
1154     <para>or</para>
1155
1156     <para>if ( this == that ) do_something();</para>
1157
1158     <para><emphasis>Note:</emphasis> The first example in "Instead of" will execute
1159     in a manner other than that which the developer desired (per
1160     indentation). Using code braces would have prevented this
1161     "feature". The "explanation" and "exception" from the point
1162     above also applies.</para>
1163
1164
1165   </sect3>
1166
1167
1168     <sect3 id="s18"><title>Do not belabor/blow-up boolean
1169     expressions</title>
1170
1171     <para><emphasis>Example:</emphasis></para>
1172 <programlisting>
1173 structure->flag = ( condition );</programlisting>
1174
1175     <para><emphasis>Instead of:</emphasis></para>
1176
1177     <para>if ( condition ) { structure->flag = 1; } else {
1178     structure->flag = 0; }</para>
1179
1180     <para><emphasis>Note:</emphasis> The former is readable and concise. The later
1181     is wordy and inefficient. Please assume that any developer new
1182     to the project has at least a "good" knowledge of C/C++. (Hope
1183     I do not offend by that last comment ... 8-)</para>
1184
1185
1186   </sect3>
1187
1188
1189     <sect3 id="s19"><title>Use white space freely because it is
1190     free</title>
1191
1192     <para><emphasis>Explanation:</emphasis></para>
1193
1194     <para>Make it readable. The notable exception to using white space
1195     freely is listed in the next guideline.</para>
1196
1197     <para><emphasis>Example:</emphasis></para>
1198 <programlisting>
1199 int first_value   = 0;
1200 int some_value    = 0;
1201 int another_value = 0;
1202 int this_variable = 0;
1203
1204 if ( this_variable == this_variable )
1205
1206 first_value = old_value + ( ( some_value - another_value ) - whatever )
1207 </programlisting>
1208   </sect3>
1209
1210
1211     <sect3 id="s20"><title>Don't use white space around structure
1212     operators</title>
1213
1214     <para><emphasis>Explanation:</emphasis></para>
1215
1216     <para>- structure pointer operator ( "->" ) - member operator (
1217     "." ) - functions and parentheses</para>
1218
1219     <para>It is a general coding practice to put pointers, references,
1220     and function parentheses next to names. With spaces, the
1221     connection between the object and variable/function name is not
1222     as clear.</para>
1223
1224     <para><emphasis>Example:</emphasis></para>
1225 <programlisting>
1226 a_struct->a_member;
1227 a_struct.a_member;
1228 function_name();</programlisting>
1229
1230     <para><emphasis>Instead of:</emphasis> a_struct -> a_member; a_struct . a_member;
1231     function_name ();</para>
1232
1233
1234   </sect3>
1235
1236
1237     <sect3 id="s21"><title>Make the last brace of a function stand
1238     out</title>
1239
1240     <para><emphasis>Example:</emphasis></para>
1241 <programlisting>
1242 int function1( ... )
1243 {
1244    ...code...
1245    return( ret_code );
1246
1247 }   /* -END- function1 */
1248
1249
1250 int function2( ... )
1251 {
1252 }   /* -END- function2 */
1253 </programlisting>
1254
1255     <para><emphasis>Instead of:</emphasis></para>
1256
1257     <para>int function1( ... ) { ...code... return( ret_code ); } int
1258     function2( ... ) { }</para>
1259
1260     <para><emphasis>Note:</emphasis> Use 1 blank line before the closing brace and 2
1261     lines afterward. This makes the end of function standout to
1262     the most casual viewer. Although function comments help
1263     separate functions, this is still a good coding practice. In
1264     fact, I follow these rules when using blocks in "for", "while",
1265     "do" loops, and long if {} statements too. After all whitespace
1266     is free!</para>
1267
1268     <para><emphasis>Status:</emphasis> developer-discretion on the number of blank
1269     lines. Enforced is the end of function comments.</para>
1270
1271
1272   </sect3>
1273
1274
1275     <sect3 id="s22"><title>Use 3 character indentions</title>
1276
1277     <para><emphasis>Explanation:</emphasis></para>
1278
1279     <para>If some use 8 character TABs and some use 3 character TABs,
1280     the code can look *very* ragged. So use 3 character indentions
1281     only. If you like to use TABs, pass your code through a filter
1282     such as "expand -t3" before checking in your code.</para>
1283
1284     <para><emphasis>Example:</emphasis></para>
1285 <programlisting>
1286 static const char * const url_code_map[256] =
1287 {
1288    NULL, ...
1289 };
1290
1291
1292 int function1( ... )
1293 {
1294    if ( 1 )
1295    {
1296       return( ALWAYS_TRUE );
1297    }
1298    else
1299    {
1300       return( HOW_DID_YOU_GET_HERE );
1301    }
1302
1303    return( NEVER_GETS_HERE );
1304
1305 }
1306 </programlisting>
1307   </sect3>
1308
1309   </sect2>
1310
1311
1312     <sect2 id="s23"><title>Initializing</title>
1313
1314
1315
1316     <sect3 id="s24"><title>Initialize all variables</title>
1317
1318     <para><emphasis>Explanation:</emphasis></para>
1319
1320     <para>Do not assume that the variables declared will not be used
1321     until after they have been assigned a value somewhere else in
1322     the code. Remove the chance of accidentally using an unassigned
1323     variable.</para>
1324
1325     <para><emphasis>Example:</emphasis></para>
1326 <programlisting>
1327 short a_short = 0;
1328 float a_float  = 0;
1329 struct *ptr = NULL;</programlisting>
1330
1331     <para><emphasis>Note:</emphasis> It is much easier to debug a SIGSEGV if the
1332     message says you are trying to access memory address 00000000
1333     and not 129FA012; or array_ptr[20] causes a SIGSEV vs.
1334     array_ptr[0].</para>
1335
1336     <para><emphasis>Status:</emphasis> developer-discretion if and only if the
1337     variable is assigned a value "shortly after" declaration.</para>
1338
1339   </sect3>
1340   </sect2>
1341
1342
1343     <sect2 id="s25"><title>Functions</title>
1344
1345
1346
1347     <sect3 id="s26"><title>Name functions that return a boolean as a
1348     question.</title>
1349
1350     <para><emphasis>Explanation:</emphasis></para>
1351
1352     <para>Value should be phrased as a question that would logically
1353     be answered as a true or false statement</para>
1354
1355     <para><emphasis>Example:</emphasis></para>
1356 <programlisting>
1357 should_we_block_this();
1358 contains_an_image();
1359 is_web_page_blank();
1360 </programlisting>
1361   </sect3>
1362
1363
1364     <sect3 id="s27"><title>Always specify a return type for a
1365     function.</title>
1366
1367     <para><emphasis>Explanation:</emphasis></para>
1368
1369     <para>The default return for a function is an int. To avoid
1370     ambiguity, create a return for a function when the return has a
1371     purpose, and create a void return type if the function does not
1372     need to return anything.</para>
1373
1374
1375   </sect3>
1376
1377
1378     <sect3 id="s28"><title>Minimize function calls when iterating by
1379     using variables</title>
1380
1381     <para><emphasis>Explanation:</emphasis></para>
1382
1383     <para>It is easy to write the following code, and a clear argument
1384     can be made that the code is easy to understand:</para>
1385
1386     <para><emphasis>Example:</emphasis></para>
1387 <programlisting>
1388 for ( size_t cnt = 0; cnt &lt; block_list_length(); cnt++ )
1389 {
1390    ....
1391 }</programlisting>
1392
1393     <para><emphasis>Note:</emphasis> Unfortunately, this makes a function call for
1394     each and every iteration. This increases the overhead in the
1395     program, because the compiler has to look up the function each
1396     time, call it, and return a value. Depending on what occurs in
1397     the block_list_length() call, it might even be creating and
1398     destroying structures with each iteration, even though in each
1399     case it is comparing "cnt" to the same value, over and over.
1400     Remember too - even a call to block_list_length() is a function
1401     call, with the same overhead.</para>
1402
1403     <para>Instead of using a function call during the iterations,
1404     assign the value to a variable, and evaluate using the
1405     variable.</para>
1406
1407     <para><emphasis>Example:</emphasis></para>
1408 <programlisting>
1409 size_t len = block_list_length();
1410
1411 for ( size_t cnt = 0; cnt &lt; len; cnt++ )
1412 {
1413    ....
1414 }</programlisting>
1415
1416     <para><emphasis>Exceptions:</emphasis> if the value of block_list_length()
1417     *may* change or could *potentially* change, then you must code the
1418     function call in the for/while loop.</para>
1419
1420
1421   </sect3>
1422
1423
1424     <sect3 id="s29"><title>Pass and Return by Const Reference</title>
1425
1426     <para><emphasis>Explanation:</emphasis></para>
1427
1428     <para>This allows a developer to define a const pointer and call
1429     your function. If your function does not have the const
1430     keyword, we may not be able to use your function. Consider
1431     strcmp, if it were defined as: extern int strcmp( char *s1,
1432     char *s2 );</para>
1433
1434     <para>I could then not use it to compare argv's in main: int main(
1435     int argc, const char *argv[] ) { strcmp( argv[0], "privoxy"
1436     ); }</para>
1437
1438     <para>Both these pointers are *const*! If the c runtime library
1439     maintainers do it, we should too.</para>
1440
1441
1442   </sect3>
1443
1444
1445     <sect3 id="s30"><title>Pass and Return by Value</title>
1446
1447     <para><emphasis>Explanation:</emphasis></para>
1448
1449     <para>Most structures cannot fit onto a normal stack entry (i.e.
1450     they are not 4 bytes or less). Aka, a function declaration
1451     like: int load_aclfile( struct client_state csp )</para>
1452
1453     <para>would not work. So, to be consistent, we should declare all
1454     prototypes with "pass by value": int load_aclfile( struct
1455     client_state *csp )</para>
1456
1457
1458   </sect3>
1459
1460
1461     <sect3 id="s31"><title>Names of include files</title>
1462
1463     <para><emphasis>Explanation:</emphasis></para>
1464
1465     <para>Your include statements should contain the file name without
1466     a path. The path should be listed in the Makefile, using -I as
1467     processor directive to search the indicated paths. An exception
1468     to this would be for some proprietary software that utilizes a
1469     partial path to distinguish their header files from system or
1470     other header files.</para>
1471
1472     <para><emphasis>Example:</emphasis></para>
1473 <programlisting>
1474 #include &lt;iostream.h&gt;     /* This is not a local include */
1475 #include "config.h"       /* This IS a local include */
1476 </programlisting>
1477
1478     <para><emphasis>Exception:</emphasis></para>
1479
1480     <para>
1481 <programlisting>
1482 /* This is not a local include, but requires a path element. */
1483 #include &lt;sys/fileName.h&gt;
1484 </programlisting>
1485 </para>
1486
1487     <para><emphasis>Note:</emphasis> Please! do not add "-I." to the Makefile
1488     without a _very_ good reason. This duplicates the #include
1489     "file.h" behavior.</para>
1490
1491
1492   </sect3>
1493
1494
1495     <sect3 id="s32"><title>Provide multiple inclusion
1496     protection</title>
1497
1498     <para><emphasis>Explanation:</emphasis></para>
1499
1500     <para>Prevents compiler and linker errors resulting from
1501     redefinition of items.</para>
1502
1503     <para>Wrap each header file with the following syntax to prevent
1504     multiple inclusions of the file. Of course, replace PROJECT_H
1505     with your file name, with "." Changed to "_", and make it
1506     uppercase.</para>
1507
1508     <para><emphasis>Example:</emphasis></para>
1509 <programlisting>
1510 #ifndef PROJECT_H_INCLUDED
1511 #define PROJECT_H_INCLUDED
1512  ...
1513 #endif /* ndef PROJECT_H_INCLUDED */
1514 </programlisting>
1515   </sect3>
1516
1517
1518     <sect3 id="s33"><title>Use `extern "C"` when appropriate</title>
1519
1520     <para><emphasis>Explanation:</emphasis></para>
1521
1522     <para>If our headers are included from C++, they must declare our
1523     functions as `extern "C"`. This has no cost in C, but increases
1524     the potential re-usability of our code.</para>
1525
1526     <para><emphasis>Example:</emphasis></para>
1527 <programlisting>
1528 #ifdef __cplusplus
1529 extern "C"
1530 {
1531 #endif /* def __cplusplus */
1532
1533 ... function definitions here ...
1534
1535 #ifdef __cplusplus
1536 }
1537 #endif /* def __cplusplus */
1538 </programlisting>
1539   </sect3>
1540
1541
1542     <sect3 id="s34"><title>Where Possible, Use Forward Struct
1543     Declaration Instead of Includes</title>
1544
1545     <para><emphasis>Explanation:</emphasis></para>
1546
1547     <para>Useful in headers that include pointers to other struct's.
1548     Modifications to excess header files may cause needless
1549     compiles.</para>
1550
1551     <para><emphasis>Example:</emphasis></para>
1552 <programlisting>
1553 /*********************************************************************
1554  * We're avoiding an include statement here!
1555  *********************************************************************/
1556 struct file_list;
1557 extern file_list *xyz;</programlisting>
1558
1559     <para><emphasis>Note:</emphasis> If you declare "file_list xyz;" (without the
1560     pointer), then including the proper header file is necessary.
1561     If you only want to prototype a pointer, however, the header
1562     file is unnecessary.</para>
1563
1564     <para><emphasis>Status:</emphasis> Use with discretion.</para>
1565
1566
1567   </sect3>
1568   </sect2>
1569
1570     <sect2 id="s35"><title>General Coding Practices</title>
1571
1572
1573
1574     <sect3 id="s36"><title>Turn on warnings</title>
1575
1576     <para><emphasis>Explanation</emphasis></para>
1577
1578     <para>Compiler warnings are meant to help you find bugs. You
1579     should turn on as many as possible. With GCC, the switch is
1580     "-Wall". Try and fix as many warnings as possible.</para>
1581
1582
1583   </sect3>
1584
1585
1586     <sect3 id="s37"><title>Provide a default case for all switch
1587     statements</title>
1588
1589     <para><emphasis>Explanation:</emphasis></para>
1590
1591     <para>What you think is guaranteed is never really guaranteed. The
1592     value that you don't think you need to check is the one that
1593     someday will be passed. So, to protect yourself from the
1594     unknown, always have a default step in a switch statement.</para>
1595
1596     <para><emphasis>Example:</emphasis></para>
1597 <programlisting>
1598 switch( hash_string( cmd ) )
1599 {
1600    case hash_actions_file :
1601       ... code ...
1602       break;
1603
1604    case hash_confdir :
1605       ... code ...
1606       break;
1607
1608    default :
1609       log_error( ... );
1610       ... anomaly code goes here ...
1611       continue; / break; / exit( 1 ); / etc ...
1612
1613 } /* end switch( hash_string( cmd ) ) */</programlisting>
1614
1615     <para><emphasis>Note:</emphasis> If you already have a default condition, you
1616     are obviously exempt from this point. Of note, most of the
1617     WIN32 code calls `DefWindowProc' after the switch statement.
1618     This API call *should* be included in a default statement.</para>
1619
1620     <para><emphasis>Another Note:</emphasis> This is not so much a readability issue
1621     as a robust programming issue. The "anomaly code goes here" may
1622     be no more than a print to the STDERR stream (as in
1623     load_config). Or it may really be an abort condition.</para>
1624
1625     <para><emphasis>Status:</emphasis> Programmer discretion is advised.</para>
1626
1627
1628   </sect3>
1629
1630
1631     <sect3 id="s38"><title>Try to avoid falling through cases in a
1632     switch statement.</title>
1633
1634     <para><emphasis>Explanation:</emphasis></para>
1635
1636     <para>In general, you will want to have a 'break' statement within
1637     each 'case' of a switch statement. This allows for the code to
1638     be more readable and understandable, and furthermore can
1639     prevent unwanted surprises if someone else later gets creative
1640     and moves the code around.</para>
1641
1642     <para>The language allows you to plan the fall through from one
1643     case statement to another simply by omitting the break
1644     statement within the case statement. This feature does have
1645     benefits, but should only be used in rare cases. In general,
1646     use a break statement for each case statement.</para>
1647
1648     <para>If you choose to allow fall through, you should comment both
1649     the fact of the fall through and reason why you felt it was
1650     necessary.</para>
1651
1652
1653   </sect3>
1654
1655
1656     <sect3 id="s39"><title>Use 'long' or 'short' Instead of
1657     'int'</title>
1658
1659     <para><emphasis>Explanation:</emphasis></para>
1660
1661     <para>On 32-bit platforms, int usually has the range of long. On
1662     16-bit platforms, int has the range of short.</para>
1663
1664     <para><emphasis>Status:</emphasis> open-to-debate. In the case of most FSF
1665     projects (including X/GNU-Emacs), there are typedefs to int4,
1666     int8, int16, (or equivalence ... I forget the exact typedefs
1667     now). Should we add these to IJB now that we have a "configure"
1668     script?</para>
1669
1670
1671   </sect3>
1672
1673
1674     <sect3 id="s40"><title>Don't mix size_t and other types</title>
1675
1676     <para><emphasis>Explanation:</emphasis></para>
1677
1678     <para>The type of size_t varies across platforms. Do not make
1679     assumptions about whether it is signed or unsigned, or about
1680     how long it is. Do not compare a size_t against another
1681     variable of a different type (or even against a constant)
1682     without casting one of the values.</para>
1683
1684
1685   </sect3>
1686
1687
1688     <sect3 id="s41"><title>Declare each variable and struct on its
1689     own line.</title>
1690
1691     <para><emphasis>Explanation:</emphasis></para>
1692
1693     <para>It can be tempting to declare a series of variables all on
1694     one line. Don't.</para>
1695
1696     <para><emphasis>Example:</emphasis></para>
1697 <programlisting>
1698 long a = 0;
1699 long b = 0;
1700 long c = 0;</programlisting>
1701
1702     <para><emphasis>Instead of:</emphasis></para>
1703
1704     <para>long a, b, c;</para>
1705
1706     <para><emphasis>Explanation:</emphasis> - there is more room for comments on the
1707     individual variables - easier to add new variables without
1708     messing up the original ones - when searching on a variable to
1709     find its type, there is less clutter to "visually"
1710     eliminate</para>
1711
1712     <para><emphasis>Exceptions:</emphasis> when you want to declare a bunch of loop
1713     variables or other trivial variables; feel free to declare them
1714     on one line. You should, although, provide a good comment on
1715     their functions.</para>
1716
1717     <para><emphasis>Status:</emphasis> developer-discretion.</para>
1718
1719
1720   </sect3>
1721
1722
1723     <sect3 id="s42"><title>Use malloc/zalloc sparingly</title>
1724
1725     <para><emphasis>Explanation:</emphasis></para>
1726
1727     <para>Create a local struct (on the stack) if the variable will
1728     live and die within the context of one function call.</para>
1729
1730     <para>Only "malloc" a struct (on the heap) if the variable's life
1731     will extend beyond the context of one function call.</para>
1732
1733     <para><emphasis>Example:</emphasis></para>
1734 <programlisting>
1735 If a function creates a struct and stores a pointer to it in a
1736 list, then it should definitely be allocated via `malloc'.
1737 </programlisting>
1738   </sect3>
1739
1740
1741     <sect3 id="s43"><title>The Programmer Who Uses 'malloc' is
1742     Responsible for Ensuring 'free'</title>
1743
1744     <para><emphasis>Explanation:</emphasis></para>
1745
1746     <para>If you have to "malloc" an instance, you are responsible for
1747     insuring that the instance is `free'd, even if the deallocation
1748     event falls within some other programmer's code. You are also
1749     responsible for ensuring that deletion is timely (i.e. not too
1750     soon, not too late). This is known as "low-coupling" and is a
1751     "good thing (tm)". You may need to offer a
1752     free/unload/destructor type function to accommodate this.</para>
1753
1754     <para><emphasis>Example:</emphasis></para>
1755 <programlisting>
1756 int load_re_filterfile( struct client_state *csp ) { ... }
1757 static void unload_re_filterfile( void *f ) { ... }</programlisting>
1758
1759     <para><emphasis>Exceptions:</emphasis></para>
1760
1761     <para>The developer cannot be expected to provide `free'ing
1762     functions for C run-time library functions ... such as
1763     `strdup'.</para>
1764
1765     <para><emphasis>Status:</emphasis> developer-discretion. The "main" use of this
1766     standard is for allocating and freeing data structures (complex
1767     or nested).</para>
1768
1769
1770   </sect3>
1771
1772
1773     <sect3 id="s44"><title>Add loaders to the `file_list' structure
1774     and in order</title>
1775
1776     <para><emphasis>Explanation:</emphasis></para>
1777
1778     <para>I have ordered all of the "blocker" file code to be in alpha
1779     order. It is easier to add/read new blockers when you expect a
1780     certain order.</para>
1781
1782     <para><emphasis>Note:</emphasis> It may appear that the alpha order is broken in
1783     places by POPUP tests coming before PCRS tests. But since
1784     POPUPs can also be referred to as KILLPOPUPs, it is clear that
1785     it should come first.</para>
1786
1787
1788   </sect3>
1789
1790
1791     <sect3 id="s45"><title>"Uncertain" new code and/or changes to
1792     existing code, use FIXME or XXX</title>
1793
1794     <para><emphasis>Explanation:</emphasis></para>
1795
1796     <para>If you have enough confidence in new code or confidence in
1797     your changes, but are not *quite* sure of the repercussions,
1798     add this:</para>
1799
1800     <para>/* FIXME: this code has a logic error on platform XYZ, *
1801     attempting to fix */ #ifdef PLATFORM ...changed code here...
1802     #endif</para>
1803
1804     <para>or:</para>
1805
1806     <para>/* FIXME: I think the original author really meant this...
1807     */ ...changed code here...</para>
1808
1809     <para>or:</para>
1810
1811     <para>/* FIXME: new code that *may* break something else... */
1812     ...new code here...</para>
1813
1814     <para><emphasis>Note:</emphasis> If you make it clear that this may or may not
1815     be a "good thing (tm)", it will be easier to identify and
1816     include in the project (or conversely exclude from the
1817     project).</para>
1818
1819
1820   </sect3>
1821
1822   </sect2>
1823
1824     <sect2 id="s46"><title>Addendum: Template for files and function
1825     comment blocks:</title>
1826
1827     <para><emphasis>Example for file comments:</emphasis></para>
1828 <programlisting>
1829 const char FILENAME_rcs[] = "$I<!-- Break CVS Substitution -->d$";
1830 /*********************************************************************
1831  *
1832  * File        :  $S<!-- Break CVS Substitution -->ource$
1833  *
1834  * Purpose     :  (Fill me in with a good description!)
1835  *
1836  * Copyright   :  Written by and Copyright (C) 2001-2009
1837  *                the Privoxy team. http://www.privoxy.org/
1838  *
1839  *                This program is free software; you can redistribute it
1840  *                and/or modify it under the terms of the GNU General
1841  *                Public License as published by the Free Software
1842  *                Foundation; either version 2 of the License, or (at
1843  *                your option) any later version.
1844  *
1845  *                This program is distributed in the hope that it will
1846  *                be useful, but WITHOUT ANY WARRANTY; without even the
1847  *                implied warranty of MERCHANTABILITY or FITNESS FOR A
1848  *                PARTICULAR PURPOSE.  See the GNU General Public
1849  *                License for more details.
1850  *
1851  *                The GNU General Public License should be included with
1852  *                this file.  If not, you can view it at
1853  *                http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
1854  *                or write to the Free Software Foundation, Inc.,
1855  *                51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 ,
1856  *                USA
1857  *
1858  *********************************************************************/
1859
1860
1861 #include "config.h"
1862
1863    ...necessary include files for us to do our work...
1864
1865 const char FILENAME_h_rcs[] = FILENAME_H_VERSION;
1866 </programlisting>
1867
1868     <para><emphasis>Note:</emphasis> This declares the rcs variables that should be
1869     added to the "show-proxy-args" page. If this is a brand new
1870     creation by you, you are free to change the "Copyright" section
1871     to represent the rights you wish to maintain.</para>
1872
1873     <para><emphasis>Note:</emphasis> The formfeed character that is present right
1874     after the comment flower box is handy for (X|GNU)Emacs users to
1875     skip the verbiage and get to the heart of the code (via
1876     `forward-page' and `backward-page'). Please include it if you
1877     can.</para>
1878
1879     <para><emphasis>Example for file header comments:</emphasis></para>
1880 <programlisting>
1881 #ifndef _FILENAME_H
1882 #define _FILENAME_H
1883 #define FILENAME_H_VERSION "$I<!-- Break CVS Substitution -->d$"
1884 /*********************************************************************
1885  *
1886  * File        :  $S<!-- Break CVS Substitution -->ource$
1887  *
1888  * Purpose     :  (Fill me in with a good description!)
1889  *
1890  * Copyright   :  Written by and Copyright (C) 2001-2009
1891  *                the Privoxy team. http://www.privoxy.org/
1892  *
1893  *                This program is free software; you can redistribute it
1894  *                and/or modify it under the terms of the GNU General
1895  *                Public License as published by the Free Software
1896  *                Foundation; either version 2 of the License, or (at
1897  *                your option) any later version.
1898  *
1899  *                This program is distributed in the hope that it will
1900  *                be useful, but WITHOUT ANY WARRANTY; without even the
1901  *                implied warranty of MERCHANTABILITY or FITNESS FOR A
1902  *                PARTICULAR PURPOSE.  See the GNU General Public
1903  *                License for more details.
1904  *
1905  *                The GNU General Public License should be included with
1906  *                this file.  If not, you can view it at
1907  *                http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
1908  *                or write to the Free Software Foundation, Inc.,
1909  *                51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 ,
1910  *                USA
1911  *
1912  *********************************************************************/
1913
1914
1915 #include "project.h"
1916
1917 #ifdef __cplusplus
1918 extern "C" {
1919 #endif
1920
1921    ... function headers here ...
1922
1923
1924 /* Revision control strings from this header and associated .c file */
1925 extern const char FILENAME_rcs[];
1926 extern const char FILENAME_h_rcs[];
1927
1928
1929 #ifdef __cplusplus
1930 } /* extern "C" */
1931 #endif
1932
1933 #endif /* ndef _FILENAME_H */
1934
1935 /*
1936   Local Variables:
1937   tab-width: 3
1938   end:
1939 */
1940 </programlisting>
1941
1942     <para><emphasis>Example for function comments:</emphasis></para>
1943 <programlisting>
1944 /*********************************************************************
1945  *
1946  * Function    :  FUNCTION_NAME
1947  *
1948  * Description :  (Fill me in with a good description!)
1949  *
1950  * parameters  :
1951  *          1  :  param1 = pointer to an important thing
1952  *          2  :  x      = pointer to something else
1953  *
1954  * Returns     :  0 => Ok, everything else is an error.
1955  *
1956  *********************************************************************/
1957 int FUNCTION_NAME( void *param1, const char *x )
1958 {
1959    ...
1960    return( 0 );
1961
1962 }
1963 </programlisting>
1964
1965     <para><emphasis>Note:</emphasis> If we all follow this practice, we should be
1966     able to parse our code to create a "self-documenting" web
1967     page.</para>
1968
1969   </sect2>
1970
1971   </sect1>
1972
1973   <!--   ~~~~~       New section      ~~~~~     -->
1974   <sect1 id="testing"><title>Testing Guidelines</title>
1975     <para>To be filled.
1976 </para>
1977
1978     <!--   ~~~~~       New section      ~~~~~     -->
1979     <sect2 id="testing-plan"><title>Testplan for releases</title>
1980       <para>
1981        Explain release numbers. major, minor. developer releases. etc.
1982
1983 <orderedlist numeration="arabic">
1984           <listitem><para>
1985 Remove any existing rpm with rpm -e
1986 </para></listitem>
1987           <listitem><para>
1988 Remove any file that was left over. This includes (but is not limited to)
1989       <itemizedlist>
1990                 <listitem><para>/var/log/privoxy</para></listitem>
1991                 <listitem><para>/etc/privoxy</para></listitem>
1992                 <listitem><para>/usr/sbin/privoxy</para></listitem>
1993                 <listitem><para>/etc/init.d/privoxy</para></listitem>
1994                 <listitem><para>/usr/doc/privoxy*</para></listitem>
1995               </itemizedlist>
1996 </para></listitem>
1997           <listitem><para>
1998 Install the rpm. Any error messages?
1999 </para></listitem>
2000           <listitem><para>start,stop,status <application>Privoxy</application> with the specific script
2001       (e.g. /etc/rc.d/init/privoxy stop). Reboot your machine. Does
2002       autostart work?</para></listitem>
2003           <listitem><para>Start browsing. Does <application>Privoxy</application> work? Logfile written?</para></listitem>
2004           <listitem><para>Remove the rpm. Any error messages? All files removed?</para></listitem>
2005         </orderedlist>
2006 </para>
2007     </sect2>
2008
2009     <!--   ~~~~~       New section      ~~~~~     -->
2010     <sect2 id="testing-report"><title>Test reports</title>
2011       <para>
2012 Please submit test reports only with the <ulink url="http://sourceforge.net/tracker/?func=add&amp;group_id=11118&amp;atid=395005">test form</ulink>
2013 at sourceforge. Three simple steps:
2014         <itemizedlist>
2015
2016           <listitem><para>Select category: the distribution you test on.</para></listitem>
2017           <listitem><para>Select group: the version of <application>Privoxy</application> that we are about to release.</para></listitem>
2018           <listitem><para>Fill the Summary and Detailed Description with something
2019               intelligent (keep it short and precise).</para>
2020           </listitem>
2021         </itemizedlist>
2022         Do not mail to the mailing list (we cannot keep track on issues there).
2023       </para>
2024     </sect2>
2025
2026   </sect1>
2027
2028   <!--   ~~~~~       New section      ~~~~~     -->
2029   <sect1 id="newrelease"><title>Releasing a New Version</title>
2030     <para>
2031         When we release versions of <application>Privoxy</application>,
2032         our work leaves our cozy secret lab and has to work in the cold
2033         RealWorld[tm]. Once it is released, there is no way to call it
2034         back, so it is very important that great care is taken to ensure
2035         that everything runs fine, and not to introduce problems in the
2036         very last minute.
2037     </para>
2038     <para>
2039         So when releasing a new version, please adhere exactly to the
2040         procedure outlined in this chapter.
2041     </para>
2042
2043     <para>
2044         The following programs are required to follow this process:
2045         <filename>ncftpput</filename> (ncftp), <filename>scp, ssh</filename> (ssh),
2046         <filename>gmake</filename> (GNU's version of make), autoconf, cvs.
2047     </para>
2048
2049     <sect2 id="versionnumbers">
2050     <title>Version numbers</title>
2051
2052     <para>
2053       First you need to determine which version number the release will have.
2054       <application>Privoxy</application> version numbers consist of three numbers,
2055       separated by dots, like in X.Y.Z (e.g. 3.0.0), where:
2056         <itemizedlist>
2057           <listitem>
2058             <para>
2059               X, the version major, is rarely ever changed. It is increased by one if
2060               turning a development branch into stable substantially changes the functionality,
2061               user interface or configuration syntax. Majors 1 and 2 were
2062               <application>Junkbuster</application>, and 3 will be the first stable
2063               <application>Privoxy</application> release.
2064             </para>
2065           </listitem>
2066           <listitem>
2067             <para>
2068               Y, the version minor, represents the branch within the major version.
2069               At any point in time, there are two branches being maintained:
2070               The stable branch, with an even minor, say, 2N, in which no functionality is
2071               being added and only bug-fixes are made, and 2N+1, the development branch, in
2072               which the further development of <application>Privoxy</application> takes
2073               place.
2074               This enables us to turn the code upside down and inside out, while at the same time
2075               providing and maintaining a stable version.
2076               The minor is reset to zero (and one) when the major is incremented. When a development
2077               branch has matured to the point where it can be turned into stable, the old stable branch
2078               2N is given up (i.e. no longer maintained), the former development branch 2N+1 becomes the
2079               new stable branch 2N+2, and a new development branch 2N+3 is opened.
2080             </para>
2081           </listitem>
2082           <listitem>
2083             <para>
2084               Z, the point or sub version, represents a release of the software within a branch.
2085               It is therefore incremented immediately before each code freeze.
2086               In development branches, only the even point versions correspond to actual releases,
2087               while the odd ones denote the evolving state of the sources on CVS in between.
2088               It follows that Z is odd on CVS in development branches most of the time. There, it gets
2089               increased to an even number immediately before a code freeze, and is increased to an odd
2090               number again immediately thereafter.
2091               This ensures that builds from CVS snapshots are easily distinguished from released versions.
2092               The point version is reset to zero when the minor changes.
2093             </para>
2094             <para>
2095               Stable branches work a little differently, since there should be
2096               little to no development happening in such branches. Remember,
2097               only bugfixes, which presumably should have had some testing
2098               before being committed. Stable branches will then have their
2099               version reported as <literal>0.0.0</literal>, during that period
2100               between releases when changes are being added. This is to denote
2101               that this code is <emphasis>not for release</emphasis>. Then
2102               as the release nears, the version is bumped according: e.g.
2103               <literal>3.0.1 -> 0.0.0 -> 3.0.2</literal>.
2104             </para>
2105           </listitem>
2106         </itemizedlist>
2107     </para>
2108     <para>
2109      In summary, the main CVS trunk is the development branch where new
2110      features are being worked on for the next stable series. This should
2111      almost always be where the most activity takes place. There is always at
2112      least one stable branch from the trunk, e.g now it is
2113      <literal>3.0</literal>, which is only used to release stable versions.
2114      Once the initial *.0 release of the stable branch has been done, then as a
2115      rule, only bugfixes that have had prior testing should be committed to
2116      the stable branch. Once there are enough bugfixes to justify a new
2117      release, the version of this branch is again incremented Example: 3.0.0
2118      -> 3.0.1 -> 3.0.2, etc are all stable releases from within the stable
2119      branch. 3.1.x is currently the main trunk, and where work on 3.2.x is
2120      taking place. If any questions, please post to the devel list
2121      <emphasis>before</emphasis> committing to a stable branch!
2122     </para>
2123     <para>
2124      Developers should remember too that if they commit a bugfix to the stable
2125      branch, this will more than likely require a separate submission to the
2126      main trunk, since these are separate development trees within CVS. If you
2127      are working on both, then this would require at least two separate check
2128      outs (i.e main trunk, <emphasis>and</emphasis> the stable release branch,
2129      which is <literal>v_3_0_branch</literal> at the moment).
2130     </para>
2131
2132     </sect2>
2133
2134     <sect2 id="beforerelease">
2135     <title>Before the Release: Freeze</title>
2136      <para>
2137        The following <emphasis>must be done by one of the
2138        developers</emphasis> prior to each new release.
2139      </para>
2140      <para>
2141       <itemizedlist>
2142        <listitem>
2143         <para>
2144          Make sure that everybody who has worked on the code in the last
2145          couple of days has had a chance to yell <quote>no!</quote> in case
2146          they have pending changes/fixes in their pipelines. Announce the
2147          freeze so that nobody will interfere with last minute changes.
2148         </para>
2149       </listitem>
2150       <listitem>
2151        <para>
2152          Increment the version number (point from odd to even in development
2153          branches!) in <filename>configure.in</filename>. (RPM spec files
2154          will need to be incremented as well.)
2155        </para>
2156       </listitem>
2157       <listitem>
2158        <para>
2159         If <filename>default.action</filename> has changed since last
2160         release (i.e. software release or standalone actions file release),
2161         bump up its version info to A.B in this line:
2162        </para>
2163        <para>
2164         <programlisting>
2165   {+add-header{X-Actions-File-Version: A.B} -filter -no-popups}
2166 </programlisting>
2167        </para>
2168        <para>
2169         Then change the version info in doc/webserver/actions/index.php,
2170         line: '$required_actions_file_version = "A.B";'
2171        </para>
2172       </listitem>
2173       <listitem>
2174        <para>
2175         All documentation should be rebuild after the version bump.
2176         Finished docs should be then be committed to CVS (for those
2177         without the ability to build these). Some docs may require
2178         rather obscure processing tools. <filename>config</filename>,
2179         the man page (and the html version of the man page), and the PDF docs
2180         fall in this category. REAMDE, the man page, AUTHORS, and config
2181         should all also be committed to CVS for other packagers. The
2182         formal docs should be uploaded to the webserver. See the
2183         Section "Updating the webserver" in this manual for details.
2184        </para>
2185       </listitem>
2186       <listitem>
2187        <para>
2188          The <citetitle>User Manual</citetitle> is also used for context
2189          sensitive help for the CGI editor. This is version sensitive, so that
2190          the user will get appropriate help for his/her release. So with
2191          each release a fresh version should be uploaded to the webserver
2192          (this is in addition to the main <citetitle>User Manual</citetitle>
2193          link from the main page since we need to keep manuals for various
2194          versions available). The CGI pages will link to something like
2195          <literal>http://privoxy.org/$(VERSION)/user-manual/</literal>. This
2196          will need to be updated for each new release. There is no Makefile
2197          target for this at this time!!! It needs to be done manually.
2198        </para>
2199       </listitem>
2200       <listitem>
2201        <para>
2202         All developers should look at the <filename>ChangeLog</filename> and
2203         make sure noteworthy changes are referenced.
2204        </para>
2205      </listitem>
2206       <listitem>
2207        <para>
2208         <emphasis>Commit all files that were changed in the above steps!</emphasis>
2209        </para>
2210       </listitem>
2211       <listitem>
2212        <para>
2213         Tag all files in CVS with the version number with
2214         <quote><command>cvs tag v_X_Y_Z</command></quote>.
2215         Don't use vX_Y_Z, ver_X_Y_Z, v_X.Y.Z (won't work) etc.
2216        </para>
2217       </listitem>
2218      <listitem>
2219        <para>
2220         If the release was in a development branch, increase the point version
2221         from even to odd (X.Y.(Z+1)) again in <filename>configure.in</filename> and
2222         commit your change.
2223        </para>
2224       </listitem>
2225      <listitem>
2226        <para>
2227         On the webserver, copy the user manual to a new top-level directory
2228         called <filename>X.Y.Z</filename>. This ensures that help links from the CGI
2229         pages, which have the version as a prefix, will go into the right version of the manual.
2230         If this is a development branch release, also symlink <filename>X.Y.(Z-1)</filename>
2231         to <filename>X.Y.Z</filename> and <filename>X.Y.(Z+1)</filename> to
2232         <filename>.</filename> (i.e. dot).
2233        </para>
2234       </listitem>
2235       </itemizedlist>
2236      </para>
2237     </sect2>
2238
2239     <sect2 id="therelease">
2240     <title>Building and Releasing the Packages</title>
2241      <para>
2242       Now the individual packages can be built and released. Note that for
2243       GPL reasons the first package to be released is always the source tarball.
2244      </para>
2245
2246      <para>
2247       For <emphasis>all</emphasis> types of packages, including the source tarball,
2248       <emphasis>you must make sure that you build from clean sources by exporting
2249       the right version from CVS into an empty directory</emphasis> (just press return when
2250       asked for a password):
2251      </para>
2252
2253      <para>
2254       <programlisting>
2255   mkdir dist # delete or choose different name if it already exists
2256   cd dist
2257   cvs -d:pserver:anonymous@ijbswa.cvs.sourceforge.net:/cvsroot/ijbswa login
2258   cvs -z3 -d:pserver:anonymous@ijbswa.cvs.sourceforge.net:/cvsroot/ijbswa export -r v_X_Y_Z current
2259 </programlisting>
2260     </para>
2261
2262     <para>
2263      <emphasis>Do NOT change</emphasis> a single bit, including, but not limited to
2264      version information after export from CVS. This is to make sure that
2265      all release packages, and with them, all future bug reports, are based
2266      on exactly the same code.
2267     </para>
2268
2269     <warning>
2270      <para>
2271       Every significant release of Privoxy has included at least one
2272       package that either had incorrect versions of files, missing files,
2273       or incidental leftovers from a previous build process that gave
2274       unknown numbers of users headaches to try to figure out what was
2275       wrong. PLEASE, make sure you are using pristene sources, and are
2276       following the prescribed process!
2277      </para>
2278     </warning>
2279
2280     <para>
2281      Please find additional instructions for the source tarball and the
2282      individual platform dependent binary packages below. And details
2283      on the Sourceforge release process below that.
2284     </para>
2285
2286     <sect3 id="pack-guidelines">
2287     <title>Note on Privoxy Packaging</title>
2288      <para>
2289       Please keep these general guidelines in mind when putting together
2290       your package. These apply to <emphasis>all</emphasis> platforms!
2291      </para>
2292      <para>
2293       <itemizedlist>
2294        <listitem>
2295         <para>
2296           <application>Privoxy</application> <emphasis>requires</emphasis>
2297           write access to: all <filename>*.action</filename> files, all
2298           logfiles, and the <filename>trust</filename> file. You will
2299           need to determine the best way to do this for your platform.
2300         </para>
2301        </listitem>
2302        <listitem>
2303         <para>
2304           Please include up to date documentation. At a bare minimum:
2305         </para>
2306         <simplelist>
2307          <member>
2308           <filename>LICENSE</filename> (top-level directory)
2309          </member>
2310         </simplelist>
2311         <simplelist>
2312          <member>
2313           <filename>README</filename> (top-level directory)
2314          </member>
2315         </simplelist>
2316         <simplelist>
2317          <member>
2318           <filename>AUTHORS</filename> (top-level directory)
2319          </member>
2320         </simplelist>
2321         <simplelist>
2322          <member>
2323           <filename>man page</filename> (top-level directory, Unix-like
2324           platforms only)
2325          </member>
2326         </simplelist>
2327         <simplelist>
2328          <member>
2329           <filename>The User Manual</filename> (doc/webserver/user-manual/)
2330          </member>
2331         </simplelist>
2332         <simplelist>
2333          <member>
2334           <filename>FAQ</filename> (doc/webserver/faq/)
2335          </member>
2336         </simplelist>
2337         <para>
2338           Also suggested: <filename>Developer Manual</filename>
2339           (doc/webserver/developer-manual) and <filename>ChangeLog</filename>
2340           (top-level directory). <filename>FAQ</filename> and the manuals are
2341           HTML docs. There are also text versions in
2342           <filename>doc/text/</filename> which could conceivably also be
2343           included.
2344         </para>
2345         <para>
2346          The documentation has been designed such that the manuals are linked
2347          to each other from parallel directories, and should be packaged
2348          that way. <filename>privoxy-index.html</filename> can also be
2349          included and can serve as a focal point for docs and other links of
2350          interest (and possibly renamed to <filename>index.html</filename>).
2351          This should be one level up from the manuals. There is a link also
2352          on this page to an HTMLized version of the man page. To avoid 404 for
2353          this, it is in CVS as
2354          <filename>doc/webserver/man-page/privoxy-man-page.html</filename>,
2355          and should be included along with the manuals. There is also a
2356          css stylesheets that can be included for better presentation:
2357          <filename>p_doc.css</filename>. This should be in the same directory
2358          with <filename>privoxy-index.html</filename>, (i.e. one level up from
2359          the manual directories).
2360         </para>
2361       </listitem>
2362       <listitem>
2363        <para>
2364         <filename>user.action</filename> and <filename>user.filter</filename>
2365         are designed for local preferences. Make sure these do not get overwritten!
2366         <filename>config</filename> should not be overwritten either. This
2367         has especially important configuration data in it.
2368         <filename>trust</filename> should be left in tact as well.
2369        </para>
2370       </listitem>
2371       <listitem>
2372        <para>
2373         Other configuration files (<filename>default.action</filename> and
2374         <filename>default.filter</filename>) should be installed as the new
2375         defaults, but all previously installed configuration files should be
2376         preserved as backups. This is just good manners :-) These files are
2377         likely to change between releases and contain important new features
2378         and bug fixes.
2379        </para>
2380      </listitem>
2381      <listitem>
2382       <para>
2383        Please check platform specific notes in this doc, if you haven't
2384        done <quote>Privoxy</quote> packaging before for other platform
2385        specific issues. Conversely, please add any notes that you know
2386        are important for your platform (or contact one of the doc
2387        maintainers to do this if you can't).
2388       </para>
2389     </listitem>
2390     <listitem>
2391      <para>
2392        Packagers should do a <quote>clean</quote> install of their
2393        package after building it. So any previous installs should be
2394        removed first to ensure the integrity of the newly built package.
2395        Then run the package for a while to make sure there are no
2396        obvious problems, before uploading.
2397      </para>
2398     </listitem>
2399
2400       </itemizedlist>
2401      </para>
2402
2403     </sect3>
2404
2405     <sect3 id="newrelease-tarball"><title>Source Tarball</title>
2406       <para>
2407         First, <emphasis>make sure that you have freshly exported the right
2408         version into an empty directory</emphasis>. (See "Building and releasing
2409         packages" above). Then run:
2410       </para>
2411       <para>
2412         <programlisting>
2413   cd current
2414   autoheader && autoconf && ./configure
2415 </programlisting>
2416       </para>
2417       <para>
2418         Then do:
2419       </para>
2420       <para>
2421         <programlisting>
2422   make tarball-dist
2423 </programlisting>
2424       </para>
2425       <para>
2426         To upload the package to Sourceforge, simply issue
2427       </para>
2428       <para>
2429         <programlisting>
2430   make tarball-upload
2431 </programlisting>
2432       </para>
2433       <para>
2434         Go to the displayed URL and release the file publicly on Sourceforge.
2435         For the change log field, use the relevant section of the
2436         <filename>ChangeLog</filename> file.
2437       </para>
2438     </sect3>
2439
2440     <sect3 id="newrelease-rpm"><title>SuSE, Conectiva or Red Hat RPM</title>
2441       <para>
2442         In following text, replace <replaceable class="parameter">dist</replaceable>
2443         with either <quote>rh</quote> for Red Hat or <quote>suse</quote> for SuSE.
2444       </para>
2445       <para>
2446         First, <emphasis>make sure that you have freshly exported the right
2447         version into an empty directory</emphasis>. (See "Building and releasing
2448         packages" above).
2449       </para>
2450       <para>
2451         As the only exception to not changing anything after export from CVS,
2452         now examine the file <filename>privoxy-</filename><replaceable class="parameter">dist</replaceable><filename>.spec</filename>
2453         and make sure that the version information and the RPM release number are
2454         correct. The RPM release numbers for each version start at one. Hence it must
2455         be reset to one if this is the first RPM for
2456         <replaceable class="parameter">dist</replaceable> which is built from version
2457         X.Y.Z. Check the
2458         <ulink url="http://sourceforge.net/project/showfiles.php?group_id=11118">file
2459         list</ulink> if unsure. Else, it must be set to the highest already available RPM
2460         release number for that version plus one.
2461       </para>
2462       <para>
2463         Then run:
2464       </para>
2465       <para>
2466         <programlisting>
2467   cd current
2468   autoheader && autoconf && ./configure
2469 </programlisting>
2470       </para>
2471       <para>
2472         Then do
2473       </para>
2474       <para>
2475         <programlisting>
2476   make <replaceable class="parameter">dist</replaceable>-dist
2477 </programlisting>
2478       </para>
2479       <para>
2480         To upload the package to Sourceforge, simply issue
2481       </para>
2482       <para>
2483         <programlisting>
2484   make <replaceable class="parameter">dist</replaceable>-upload <replaceable class="parameter">rpm_packagerev</replaceable>
2485 </programlisting>
2486       </para>
2487       <para>
2488         where <replaceable class="parameter">rpm_packagerev</replaceable> is the
2489         RPM release number as determined above.
2490         Go to the displayed URL and release the file publicly on Sourceforge.
2491         Use the release notes and change log from the source tarball package.
2492       </para>
2493     </sect3>
2494
2495     <sect3 id="newrelease-os2"><title>OS/2</title>
2496       <para>
2497         First, <emphasis>make sure that you have freshly exported the right
2498         version into an empty directory</emphasis>. (See "Building and releasing
2499         packages" above). Then get the OS/2 Setup module:
2500       </para>
2501       <para>
2502         <programlisting>
2503   cvs -z3 -d:pserver:anonymous@ijbswa.cvs.sourceforge.net:/cvsroot/ijbswa co os2setup
2504 </programlisting>
2505       </para>
2506       <para>
2507         You will need a mix of development tools.
2508         The main compilation takes place with IBM Visual Age C++.
2509         Some ancillary work takes place with GNU tools, available from
2510         various sources like hobbes.nmsu.edu.
2511         Specificially, you will need <filename>autoheader</filename>,
2512         <filename>autoconf</filename> and <filename>sh</filename> tools.
2513         The packaging takes place with WarpIN, available from various sources, including
2514         its home page: <ulink url="http://www.xworkplace.org/">xworkplace</ulink>.
2515       </para>
2516       <para>
2517         Change directory to the <filename>os2setup</filename> directory.
2518         Edit the os2build.cmd file to set the final executable filename.
2519         For example,
2520       </para>
2521       <para>
2522         <programlisting>
2523   installExeName='privoxyos2_setup_X.Y.Z.exe'
2524 </programlisting>
2525       </para>
2526       <para>
2527         Next, edit the <filename>IJB.wis</filename> file so the release number matches
2528         in the <filename>PACKAGEID</filename> section:
2529       </para>
2530       <para>
2531         <programlisting>
2532   PACKAGEID="Privoxy Team\Privoxy\Privoxy Package\X\Y\Z"
2533 </programlisting>
2534       </para>
2535       <para>
2536         You're now ready to build.  Run:
2537       </para>
2538       <para>
2539         <programlisting>
2540   os2build
2541 </programlisting>
2542       </para>
2543       <para>
2544          You will find the  WarpIN-installable executable in the
2545         <filename>./files</filename> directory. Upload this anonymously to
2546          <filename>uploads.sourceforge.net/incoming</filename>, create a release
2547          for it, and you're done. Use the release notes and Change Log from the
2548          source tarball package.
2549       </para>
2550     </sect3>
2551
2552     <sect3 id="newrelease-solaris"><title>Solaris</title>
2553       <para>
2554         Login to Sourceforge's compilefarm via ssh:
2555       </para>
2556       <para>
2557         <programlisting>
2558   ssh cf.sourceforge.net
2559 </programlisting>
2560       </para>
2561       <para>
2562         Choose the right operating system (not the Debian one).
2563         When logged in, <emphasis>make sure that you have freshly exported the right
2564         version into an empty directory</emphasis>. (See "Building and releasing
2565         packages" above). Then run:
2566       </para>
2567       <para>
2568         <programlisting>
2569   cd current
2570   autoheader && autoconf && ./configure
2571 </programlisting>
2572       </para>
2573       <para>
2574         Then run
2575       </para>
2576       <para>
2577         <programlisting>
2578   gmake solaris-dist
2579 </programlisting>
2580       </para>
2581       <para>
2582         which creates a gzip'ed tar archive. Sadly, you cannot use <command>make
2583         solaris-upload</command> on the Sourceforge machine (no ncftpput). You now have
2584         to manually upload the archive to Sourceforge's ftp server and release
2585         the file publicly. Use the release notes and Change Log from the
2586         source tarball package.
2587       </para>
2588     </sect3>
2589
2590     <sect3 id="newrelease-windows"><title>Windows</title>
2591       <para>
2592         You should ensure you have the latest version of Cygwin (from
2593         <ulink url="http://www.cygwin.com/">http://www.cygwin.com/</ulink>).
2594         Run the following commands from within a Cygwin bash shell.
2595       </para>
2596       <para>
2597         First, <emphasis>make sure that you have freshly exported the right
2598         version into an empty directory</emphasis>. (See "Building and releasing
2599         packages" above). Then get the Windows setup module:
2600       </para>
2601       <para>
2602       <programlisting>
2603   cvs -z3  -d:pserver:anonymous@ijbswa.cvs.sourceforge.net:/cvsroot/ijbswa co winsetup
2604 </programlisting>
2605       </para>
2606       <para>
2607         Then you can build the package.  This is fully automated, and is
2608         controlled by <filename>winsetup/GNUmakefile</filename>.
2609         All you need to do is:
2610       </para>
2611       <para>
2612       <programlisting>
2613   cd winsetup
2614   make
2615 </programlisting>
2616       </para>
2617       <para>
2618         Now you can manually rename <filename>privoxy_setup.exe</filename> to
2619         <filename>privoxy_setup_X_Y_Z.exe</filename>, and upload it to
2620         SourceForge. When releasing the package on SourceForge, use the release notes
2621         and Change Log from the source tarball package.
2622       </para>
2623     </sect3>
2624
2625     <sect3 id="newrelease-debian"><title>Debian</title>
2626       <para>
2627         First, <emphasis>make sure that you have freshly exported the
2628         right version into an empty directory</emphasis>. (See
2629         "Building and releasing packages" above).  Then add a log
2630         entry to <filename>debian/changelog</filename>, if it is not
2631         already there, for example by running:
2632       </para>
2633       <para>
2634         <programlisting>
2635   debchange -v &p-version;-&p-status;-1 "New upstream version"
2636 </programlisting>
2637       </para>
2638       <para>
2639         Then, run:
2640       </para>
2641       <para>
2642         <programlisting>
2643   dpkg-buildpackage -rfakeroot -us -uc -b
2644 </programlisting>
2645       </para>
2646       <para>
2647         This will create
2648         <filename>../privoxy_&p-version;-&p-status;-1_i386.deb</filename>
2649         which can be uploaded.  To upload the package to Sourceforge, simply
2650         issue
2651       </para>
2652       <para>
2653         <programlisting>
2654   make debian-upload
2655 </programlisting>
2656       </para>
2657     </sect3>
2658
2659     <sect3 id="newrelease-macosx"><title>Mac OS X</title>
2660       <para>
2661         First, <emphasis>make sure that you have freshly exported the right
2662         version into an empty directory</emphasis>. (See "Building and releasing
2663         packages" above).
2664       </para>
2665       <para>
2666         There are three modules available in the CVS repository for use on Mac
2667         OS X, though technically only two of them generate a release (the other
2668         can be used to install from source).
2669       </para>
2670       <sect4 id="OS-X-OSXPackageBuilder-module">
2671       <title>OSXPackageBuilder module</title>
2672         <para>
2673           The OSXPackageBuilder module generates OS X installer packages
2674           supporting all Macs running OS X 10.4 and above. Obtain it from CVS as
2675           follows into a folder parallel to the exported privoxy source:
2676           <programlisting>
2677   cvs -z3 -d:pserver:anonymous@ijbswa.cvs.sourceforge.net:/cvsroot/ijbswa co OSXPackageBuilder
2678 </programlisting>
2679         </para>
2680         <para>
2681           The module contains complete instructions on its usage in the file
2682           <filename>OS X Package Builder HOWTO.txt</filename>.
2683         </para>
2684         <para>
2685           Once the package(s) have been generated, you can then upload them
2686           directly to the Files section of the Sourceforge project in the
2687           Macintosh (OS X) folder. Each new version release of Privoxy should
2688           have a new subfolder created in which to store its files. Please
2689           ensure that the folder contains a readme file that makes it clear
2690           which package is for whichversion of OS X.
2691         </para>
2692       </sect4>
2693       <sect4 id="OS-X-osxsetup-module">
2694       <title>osxsetup module (DEPRECATED)</title>
2695         <para>
2696           <emphasis>This module is deprecated since the installer it generates
2697           places all Privoxy files in one folder in a non-standard location, and
2698           supports only Intel Macs running OS X 10.6 or higher.</emphasis>
2699         </para>
2700         <para>
2701           Check out the module from CVS as follows into a folder parallel to the
2702           exported privoxy source:
2703           <programlisting>
2704   cvs -z3 -d:pserver:anonymous@ijbswa.cvs.sourceforge.net:/cvsroot/ijbswa co osxsetup
2705 </programlisting>
2706         </para>
2707         <para>
2708           Then run:
2709         </para>
2710         <para>
2711           <programlisting>
2712   cd osxsetup
2713   build
2714 </programlisting>
2715         </para>
2716         <para>
2717           This will run <filename>autoheader</filename>, <filename>autoconf</filename>
2718           and <filename>configure</filename> as well as <filename>make</filename>.
2719           Finally, it will copy over the necessary files to the ./osxsetup/files
2720           directory for further processing by <filename>PackageMaker</filename>.
2721         </para>
2722         <para>
2723         Bring up PackageMaker with the PrivoxyPackage.pmsp definition file,
2724         modify the package name to match the release, and hit the "Create
2725         package" button. If you specify ./Privoxy.pkg as the output package
2726         name, you can then create the distributable zip file with the command:
2727         </para>
2728         <para>
2729           <programlisting>
2730   zip -r privoxyosx_setup_x.y.z.zip Privoxy.pkg
2731 </programlisting>
2732         </para>
2733         <para>
2734           You can then upload this file directly to the Files section of the
2735           Sourceforge project in the Macintosh (OS X) folder. Each new version
2736           release of Privoxy should have a new subfolder created in which to
2737           store its files.
2738           Please ensure that the folder contains a readme file that makes it
2739           clear which version(s) of OS X the package supports.
2740         </para>
2741       </sect4>
2742       <sect4 id="OS-X-macsetup-module">
2743       <title>macsetup module</title>
2744         <para>
2745           The macsetup module is ideal if you wish to build and install Privoxy
2746           from source on a single machine.
2747         </para>
2748         <para>
2749           Check out the module from CVS as follows into a folder parallel to the
2750           exported privoxy source:
2751           <programlisting>
2752   cvs -z3 -d:pserver:anonymous@ijbswa.cvs.sourceforge.net:/cvsroot/ijbswa co macsetup
2753 </programlisting>
2754         </para>
2755         <para>
2756           The module contains complete instructions on its usage in its
2757           <filename>README</filename> file. The end result will be the the
2758           exported version of Privoxy installed on the build machine.
2759         </para>
2760       </sect4>
2761     </sect3>
2762
2763     <sect3 id="newrelease-freebsd"><title>FreeBSD</title>
2764       <para>
2765         Login to Sourceforge's compile-farm via ssh:
2766       </para>
2767       <para>
2768         <programlisting>
2769   ssh cf.sourceforge.net
2770 </programlisting>
2771       </para>
2772       <para>
2773         Choose the right operating system.
2774         When logged in, <emphasis>make sure that you have freshly exported the right
2775         version into an empty directory</emphasis>. (See "Building and releasing
2776         packages" above). Then run:
2777       </para>
2778       <para>
2779         <programlisting>
2780   cd current
2781   autoheader && autoconf && ./configure
2782 </programlisting>
2783       </para>
2784       <para>
2785         Then run:
2786       </para>
2787       <para>
2788         <programlisting>
2789   gmake freebsd-dist
2790 </programlisting>
2791       </para>
2792       <para>
2793         which creates a gzip'ed tar archive. Sadly, you cannot use <command>make
2794         freebsd-upload</command> on the Sourceforge machine (no ncftpput). You now have
2795         to manually upload the archive to Sourceforge's ftp server and release
2796         the file publicly. Use the release notes and Change Log from the
2797         source tarball package.
2798       </para>
2799     </sect3>
2800
2801     <sect3 id="newrelease-hpux"><title>HP-UX 11</title>
2802       <para>
2803         First, <emphasis>make sure that you have freshly exported the right
2804         version into an empty directory</emphasis>. (See "Building and releasing
2805         packages" above). Then run:
2806       </para>
2807       <para>
2808         <programlisting>
2809   cd current
2810   autoheader && autoconf && ./configure
2811 </programlisting>
2812       </para>
2813       <para>
2814         Then do FIXME.
2815       </para>
2816     </sect3>
2817
2818     <sect3 id="newrelease-amiga"><title>Amiga OS</title>
2819       <para>
2820         First, <emphasis>make sure that you have freshly exported the right
2821         version into an empty directory</emphasis>. (See "Building and releasing
2822         packages" above). Then run:
2823       </para>
2824       <para>
2825         <programlisting>
2826   cd current
2827   autoheader && autoconf && ./configure
2828 </programlisting>
2829       </para>
2830       <para>
2831         Then do FIXME.
2832       </para>
2833     </sect3>
2834
2835     <sect3 id="newrelease-aix"><title>AIX</title>
2836       <para>
2837         Login to Sourceforge's compilefarm via ssh:
2838       </para>
2839       <para>
2840         <programlisting>
2841   ssh cf.sourceforge.net
2842 </programlisting>
2843       </para>
2844       <para>
2845         Choose the right operating system.
2846         When logged in, <emphasis>make sure that you have freshly exported the right
2847         version into an empty directory</emphasis>. (See "Building and releasing
2848         packages" above). Then run:
2849       </para>
2850       <para>
2851         <programlisting>
2852   cd current
2853   autoheader && autoconf && ./configure
2854 </programlisting>
2855       </para>
2856       <para>
2857         Then run:
2858       </para>
2859       <para>
2860         <programlisting>
2861   make aix-dist
2862 </programlisting>
2863       </para>
2864       <para>
2865         which creates a gzip'ed tar archive. Sadly, you cannot use <command>make
2866         aix-upload</command> on the Sourceforge machine (no ncftpput). You now have
2867         to manually upload the archive to Sourceforge's ftp server and release
2868         the file publicly. Use the release notes and Change Log from the
2869         source tarball package.
2870       </para>
2871     </sect3>
2872    </sect2>
2873
2874    <sect2 id="releasing">
2875    <title>Uploading and Releasing Your Package</title>
2876     <para>
2877       After the package is ready, it is time to upload it
2878       to SourceForge, and go through the release steps. The upload
2879       is done via FTP:
2880     </para>
2881      <para>
2882       <itemizedlist>
2883        <listitem>
2884         <para>
2885           Upload to: <ulink url="ftp://upload.sourceforge.net/incoming">ftp://upload.sourceforge.net/incoming</ulink>
2886         </para>
2887       </listitem>
2888       <listitem>
2889        <para>
2890          user: <literal>anonymous</literal>
2891        </para>
2892       </listitem>
2893       <listitem>
2894        <para>
2895          password: <literal>ijbswa-developers@lists.sourceforge.net</literal>
2896        </para>
2897       </listitem>
2898      </itemizedlist>
2899     </para>
2900     <para>
2901      Or use the <command>make</command> targets as described above.
2902     </para>
2903     <para>
2904      Once this done go to <ulink
2905       url="https://sourceforge.net/project/admin/editpackages.php?group_id=11118"
2906       >https://sourceforge.net/project/admin/editpackages.php?group_id=11118</ulink>,
2907      making sure you are logged in. Find your target platform in the
2908      second column, and click <literal>Add Release</literal>. You will
2909      then need to create a new release for your package, using the format
2910      of <literal>$VERSION ($CODE_STATUS)</literal>, e.g. <emphasis>&p-version;
2911      (beta)</emphasis>.
2912     </para>
2913     <para>
2914      Now just follow the prompts. Be sure to add any appropriate Release
2915      notes. You should see your freshly uploaded packages in
2916      <quote>Step 2. Add Files To This Release</quote>. Check the
2917      appropriate box(es). Remember at each step to hit the
2918      <quote>Refresh/Submit</quote> buttons! You should now see your
2919      file(s) listed in Step 3. Fill out the forms with the appropriate
2920      information for your platform, being sure to hit <quote>Update</quote>
2921      for each file. If anyone is monitoring your platform, check the
2922      <quote>email</quote> box at the very bottom to notify them of
2923      the new package. This should do it!
2924     </para>
2925     <para>
2926      If you have made errors, or need to make changes, you can go through
2927      essentially the same steps, but select <literal>Edit Release</literal>,
2928      instead of <literal>Add Release</literal>.
2929     </para>
2930    </sect2>
2931
2932     <sect2 id="afterrelease">
2933     <title>After the Release</title>
2934      <para>
2935       When all (or: most of the) packages have been uploaded and made available,
2936       send an email to the <ulink url="mailto:ijbswa-announce@lists.sourceforge.net">announce
2937       mailing list</ulink>, Subject: "Version X.Y.Z available for download". Be sure to
2938       include the
2939       <ulink url="http://sourceforge.net/project/showfiles.php?group_id=11118">download
2940       location</ulink>, the release notes and the Changelog. Also, post an
2941       updated News item on the project page Sourceforge, and update the Home
2942       page and docs linked from the Home page (see below). Other news sites
2943       and release oriented sites, such as Freshmeat, should also be notified.
2944      </para>
2945    </sect2>
2946
2947   </sect1>
2948
2949   <!--   ~~~~~       New section      ~~~~~     -->
2950   <sect1 id="webserver-update"><title>Update the Webserver</title>
2951    <para>
2952     The webserver should be updated at least with each stable release. When
2953     updating, please follow these steps to make sure that no broken links,
2954     inconsistent contents or permission problems will occur (as it has many
2955     times in the past!):
2956    </para>
2957    <para>
2958     If you have changed anything in the stable-branch documentation source
2959     SGML files, do:
2960    </para>
2961    <para>
2962     <programlisting>
2963   make dok dok-pdf # (or 'make redhat-dok dok-pdf' if 'make dok' doesn't work for you)
2964 </programlisting>
2965    </para>
2966    <para>
2967     That will generate <filename>doc/webserver/user-manual</filename>,
2968     <filename>doc/webserver/developer-manual</filename>,
2969     <filename>doc/webserver/faq</filename>,
2970     <filename>doc/pdf/*.pdf</filename> and
2971     <filename>doc/webserver/index.html</filename> automatically.
2972    </para>
2973    <para>
2974     If you changed the manual page sources, generate
2975     <filename>doc/webserver/man-page/privoxy-man-page.html</filename>
2976     by running <quote><command>make man</command></quote>. (This is
2977     a separate target due to dependencies on some obscure perl scripts
2978     [now in CVS, but not well tested]. See comments in <filename>GNUmakefile</filename>.)
2979    </para>
2980    <para>
2981     If you want to add new files to the webserver, create them locally in
2982     the <filename>doc/webserver/*</filename> directory (or
2983     create new directories under <filename>doc/webserver</filename>).
2984    </para>
2985    <para>
2986     Next, commit any changes from the above steps to CVS. All set?
2987     If these are docs in the stable branch, then do:
2988    </para>
2989    <para>
2990     <programlisting>
2991   make webserver
2992 </programlisting>
2993    </para>
2994    <para>
2995     This will do the upload to <ulink url="http://www.privoxy.org/">the
2996     webserver</ulink> (www.privoxy.org) and ensure all files and directories
2997     there are group writable.
2998    </para>
2999    <para>
3000     Please do <emphasis>NOT</emphasis> use any other means of transferring
3001     files to the webserver to avoid permission problems. Also, please do not
3002     upload docs from development branches or versions. The publicly posted
3003     docs should be in sync with the last official release.
3004    </para>
3005   </sect1>
3006
3007   <!--   ~~~~~       New section      ~~~~~     -->
3008   <sect1 id="contact"><title>Contacting the developers, Bug Reporting and Feature Requests</title>
3009 <!-- Include contacting.sgml  -->
3010  &contacting;
3011 <!-- end contacting -->
3012   </sect1>
3013
3014
3015 <!--   ~~~~~~~~       New section Header    ~~~~~~~~~     -->
3016 <sect1 id="copyright"><title>Privoxy Copyright, License and History</title>
3017
3018 <!-- Include copyright.sgml -->
3019  &copyright;
3020 <!-- end -->
3021
3022 <!--   ~~~~~       New section      ~~~~~     -->
3023 <sect2><title>License</title>
3024 <!-- Include copyright.sgml: -->
3025  &license;
3026 <!-- end copyright -->
3027 </sect2>
3028 <!--  ~  End section  ~  -->
3029
3030 <!--   ~~~~~       New section      ~~~~~     -->
3031 <sect2><title>History</title>
3032 <!-- Include history.sgml -->
3033  &history;
3034 <!-- end -->
3035 </sect2>
3036
3037 </sect1>
3038
3039   <!--   ~~~~~       New section      ~~~~~     -->
3040   <sect1 id="seealso"><title>See also</title>
3041 <!-- Include seealso.sgml -->
3042  &seealso;
3043 <!-- end  -->
3044
3045   </sect1>
3046
3047   <!--
3048
3049   This program is free software; you can redistribute it
3050   and/or modify it under the terms of the GNU General
3051   Public License as published by the Free Software
3052   Foundation; either version 2 of the License, or (at
3053   your option) any later version.
3054
3055   This program is distributed in the hope that it will
3056   be useful, but WITHOUT ANY WARRANTY; without even the
3057   implied warranty of MERCHANTABILITY or FITNESS FOR A
3058   PARTICULAR PURPOSE.  See the GNU General Public
3059   License for more details.
3060
3061   The GNU General Public License should be included with
3062   this file.  If not, you can view it at
3063   http://www.gnu.org/copyleft/gpl.html
3064   or write to the Free Software Foundation, Inc., 59
3065   Temple Place - Suite 330, Boston, MA  02111-1307, USA.
3066
3067   $Log: developer-manual.sgml,v $
3068   Revision 2.43  2012/03/20 13:04:03  fabiankeil
3069   Comment out references to multiple branches
3070
3071   We currently don't use any.
3072
3073   Revision 2.42  2012/03/20 13:03:05  fabiankeil
3074   Bump copyright date
3075
3076   Revision 2.41  2012/03/19 12:56:08  fabiankeil
3077   Untabify
3078
3079   Revision 2.40  2012/03/18 15:41:49  fabiankeil
3080   Bump entities to 3.0.20 UNRELEASED
3081
3082   Revision 2.39  2012/03/18 01:16:35  diem
3083   Brought OS X section up to date, deprecating the osxsetup module and adding a section referring to the OSXPackageBuilder module
3084
3085   Revision 2.38  2011/12/26 17:05:40  fabiankeil
3086   Bump entities for 3.0.19
3087
3088   Revision 2.37  2011/11/13 17:03:54  fabiankeil
3089   Bump entities for 3.0.18 stable
3090
3091   Revision 2.36  2011/09/04 11:10:12  fabiankeil
3092   Ditch trailing whitespace
3093
3094   Revision 2.35  2011/08/17 10:40:07  fabiankeil
3095   Update the entities.
3096
3097   This commit is chronological out of order.
3098
3099   Revision 2.34  2010/11/06 12:55:48  fabiankeil
3100   Set p-version to 3.0.17
3101
3102   Revision 2.33  2010/02/13 17:38:27  fabiankeil
3103   Update entities for 3.0.16 stable.
3104
3105   Revision 2.32  2009/11/15 14:24:12  fabiankeil
3106   Prepare to generate docs for 3.0.16 UNRELEASED.
3107
3108   Revision 2.31  2009/10/10 05:48:55  fabiankeil
3109   Prepare for 3.0.15 beta.
3110
3111   Revision 2.30  2009/07/18 16:24:39  fabiankeil
3112   Bump entities for 3.0.14 beta.
3113
3114   Revision 2.29  2009/06/12 14:30:58  fabiankeil
3115   Update entities for 3.0.13 beta.
3116
3117   Revision 2.28  2009/05/16 13:27:21  fabiankeil
3118   Remove CVS revision logs. TODO item #33.
3119
3120   Revision 2.27  2009/02/19 02:20:22  hal9
3121   Make some links in seealso conditional. Man page is now privoxy only links.
3122
3123   Revision 2.26  2009/02/12 16:08:26  fabiankeil
3124   Declare the code stable.
3125
3126   Revision 2.25  2009/02/12 15:37:05  fabiankeil
3127   Update templates.
3128
3129   Revision 2.24  2009/01/13 16:50:35  fabiankeil
3130   The standard.action file is gone.
3131
3132   Revision 2.23  2008/09/26 17:02:01  fabiankeil
3133   - Break some more CVS substitutions in examples.
3134   - Remove Junkbusters reference in example header
3135     for new files.
3136
3137   Revision 2.22  2008/08/30 15:37:35  fabiankeil
3138   Update entities.
3139
3140   Revision 2.21  2008/08/16 08:51:28  fabiankeil
3141   Update version-related entities.
3142
3143   Revision 2.20  2008/06/14 13:21:24  fabiankeil
3144   Prepare for the upcoming 3.0.9 beta release.
3145
3146   Revision 2.19  2008/05/12 11:13:33  fabiankeil
3147   Clarify that Privoxy is licensed under GPL version 2.
3148
3149   Revision 2.18  2008/02/04 12:14:06  fabiankeil
3150   Change "Edit Packages" URL to use https.
3151
3152   Revision 2.17  2008/02/03 21:37:41  hal9
3153   Apply patch from Mark: s/OSX/OS X/
3154
3155   Revision 2.16  2008/01/19 17:52:38  hal9
3156   Re-commit to fix various minor issues for new release.
3157
3158   Revision 2.15  2008/01/19 15:03:05  hal9
3159   Doc sources tagged for 3.0.8 release.
3160
3161   Revision 2.14  2008/01/17 01:49:51  hal9
3162   Change copyright notice for docs s/2007/2008/. All these will be rebuilt soon
3163   enough.
3164
3165   Revision 2.13  2007/10/30 17:59:31  fabiankeil
3166   - Bump p-version, p-status and copyright date.
3167   - Mention that the manual is out of date.
3168   - Don't use examples with HardToReadCamelCase after
3169     explaining that we actually don't like that.
3170   - Minor cosmetics.
3171
3172   Revision 2.12  2006/11/14 01:57:46  hal9
3173   Dump all docs prior to 3.0.6 release. Various minor changes to faq and user
3174   manual.
3175
3176   Revision 2.11  2006/09/26 02:36:29  hal9
3177   Fix broken link per bug tracker.
3178
3179   Revision 2.10  2006/09/22 01:27:55  hal9
3180   Final commit of probably various minor changes here and there. Unless
3181   something changes this should be ready for pending release.
3182
3183   Revision 2.9  2006/09/14 02:30:07  hal9
3184   Fix ijbswa cvs links. Update notes on release process, and which config files
3185   should be overwritten and which not.
3186
3187   Revision 2.8  2006/08/22 23:35:01  hal9
3188   Fix email address, cvs URI, address branching changes and various other
3189   small updates.
3190
3191   Revision 2.7  2006/07/18 14:48:50  david__schmidt
3192   Reorganizing the repository: swapping out what was HEAD (the old 3.1 branch)
3193   with what was really the latest development (the v_3_0_branch branch)
3194
3195   Revision 1.46.2.11  2002/12/11 13:12:15  hal9
3196   Rewrite cvs write access give-away section.
3197
3198   Revision 1.46.2.10  2002/09/26 21:53:45  hal9
3199   Changes to reflect recent change in stable branch commit policy (hopefully
3200   clearer now).
3201
3202   Revision 1.46.2.9  2002/09/26 01:21:40  hal9
3203   Porting 3.1.1 changes: more on cvs and branches, more on versions and
3204   releases.
3205
3206   Revision 1.46.2.8  2002/08/17 00:16:10  hal9
3207   Add note on updating webserver for User-manual/CGI editor, which is version
3208   dependent (and different from main UM link).
3209
3210   Revision 1.46.2.7  2002/08/14 17:29:25  hal9
3211   Add small notes on post-release steps, and uploading docs to webserver.
3212
3213   Revision 1.46.2.6  2002/08/10 11:40:25  oes
3214   Added disclaimer about probably being out-of-date and two small hints
3215
3216   Revision 1.46.2.5  2002/08/09 01:15:12  hal9
3217   Added some notes on pre-release steps (test builds first, update ChangeLog).
3218
3219   Revision 1.46.2.4  2002/05/29 00:30:59  mal0rd
3220   Fixed a little formatting.  Clarified debian section.
3221
3222   Revision 1.46.2.3  2002/05/28 04:32:45  hal9
3223   Change hints on bundling index.html to privoxy-index.html
3224
3225   Revision 1.46.2.2  2002/05/26 17:04:24  hal9
3226   -Spellcheck, very minor edits, and sync across branches
3227
3228   Revision 1.48  2002/05/26 12:48:31  roro
3229   Add releasing information about Debian.
3230
3231   Revision 1.47  2002/05/26 04:55:11  mal0rd
3232   Added debian-dist and debian-upload targets.  Also documented usage.
3233
3234   Revision 1.46  2002/05/22 17:15:00  oes
3235   Updated intro
3236
3237   Revision 1.45  2002/05/19 23:01:54  hal9
3238   Add small section on general packaging guidelines (e.g. actions files must
3239   be writable).
3240
3241   Revision 1.44  2002/05/15 03:55:17  hal9
3242   Fix ulink -> link, and minor modification to release process section for
3243   clarification.
3244
3245   Revision 1.43  2002/05/10 01:48:19  hal9
3246   This is mostly proposed copyright/licensing additions and changes. Docs
3247   are still GPL, but licensing and copyright are more visible. Also, copyright
3248   changed in doc header comments (eliminate references to JB except FAQ).
3249
3250   Revision 1.42  2002/05/05 20:26:02  hal9
3251   Sorting out license vs copyright in these docs.
3252
3253   Revision 1.41  2002/05/04 08:44:44  swa
3254   bumped version
3255
3256   Revision 1.40  2002/05/04 00:43:43  hal9
3257   -Remove TOC/first page kludge with proper stylesheet fix.
3258   -Combined the two very brief sections: Intro and Quickstart.
3259
3260   Revision 1.39  2002/05/02 15:08:25  oes
3261   Added explanation about version numbers and RPM package revisions
3262
3263   Revision 1.38  2002/04/29 02:20:31  hal9
3264   Add info on steps for uploading and the release process on SF.
3265
3266   Revision 1.37  2002/04/26 17:23:29  swa
3267   bookmarks cleaned, changed structure of user manual, screen and programlisting cleanups, and numerous other changes that I forgot
3268
3269   Revision 1.36  2002/04/26 05:25:23  hal9
3270   Mass commit to catch a few scattered fixes.
3271
3272   Revision 1.35  2002/04/17 15:16:15  oes
3273   Added link to docbook crash course
3274
3275   Revision 1.34  2002/04/15 23:39:32  oes
3276    - Extended & fixed the release section
3277    - Added CVS guideline sections
3278    - Separated webserver section from release section
3279    - Commented out boilerplate inclusion (If you don't know yet what it is,
3280      you shouldn't mess with its code ;-)
3281    - Nits & fixes
3282
3283   Revision 1.33  2002/04/12 03:49:53  hal9
3284   Spell checked. Clarification on where docs are kept.
3285
3286   Revision 1.32  2002/04/11 21:29:58  jongfoster
3287   Documenting Win32 release procedure
3288
3289   Revision 1.31  2002/04/11 09:32:52  oes
3290   more nits
3291
3292   Revision 1.30  2002/04/11 09:24:53  oes
3293   nits
3294
3295   Revision 1.29  2002/04/10 18:45:14  swa
3296   generated
3297
3298   Revision 1.28  2002/04/08 22:59:26  hal9
3299   Version update. Spell chkconfig correctly :)
3300
3301   Revision 1.27  2002/04/08 15:31:18  hal9
3302   Touch ups to documentation section.
3303
3304   Revision 1.26  2002/04/07 23:50:08  hal9
3305   Documentation changes to reflect HTML docs now in CVS, and new generated files
3306   list.
3307
3308   Revision 1.25  2002/04/06 05:07:28  hal9
3309   -Add privoxy-man-page.sgml, for man page.
3310   -Add authors.sgml for AUTHORS (and p-authors.sgml)
3311   -Reworked various aspects of various docs.
3312   -Added additional comments to sub-docs.
3313
3314   Revision 1.24  2002/04/04 21:33:37  hal9
3315   More on documenting the documents.
3316
3317   Revision 1.23  2002/04/04 18:46:47  swa
3318   consistent look. reuse of copyright, history et. al.
3319
3320   Revision 1.22  2002/04/04 17:27:56  swa
3321   more single file to be included at multiple points. make maintaining easier
3322
3323   Revision 1.21  2002/04/04 06:48:37  hal9
3324   Structural changes to allow for conditional inclusion/exclusion of content
3325   based on entity toggles, e.g. 'entity % p-not-stable  "INCLUDE"'. And
3326   definition of internal entities, e.g. 'entity p-version "2.9.13"' that will
3327   eventually be set by Makefile.
3328   More boilerplate text for use across multiple docs.
3329
3330   Revision 1.20  2002/04/04 03:28:27  david__schmidt
3331   Add Mac OS X section
3332
3333   Revision 1.19  2002/04/03 15:09:42  david__schmidt
3334   Add OS/2 build section
3335
3336   Revision 1.18  2002/04/03 03:51:48  hal9
3337   Touch ups.
3338
3339   Revision 1.17  2002/04/03 01:21:17  hal9
3340   Implementing Andreas's suggestions for Release sections.
3341
3342   Revision 1.16  2002/03/31 23:04:40  hal9
3343   Fleshed out the doc section, and added something for an intro so it was not
3344   blank.
3345
3346   Revision 1.15  2002/03/30 22:29:47  swa
3347   wrong make flavour
3348
3349   Revision 1.14  2002/03/30 19:04:08  swa
3350   people release differently. no good.
3351   I want to make parts of the docs only.
3352
3353   Revision 1.13  2002/03/27 01:16:41  hal9
3354   ditto
3355
3356   Revision 1.12  2002/03/27 01:02:51  hal9
3357   Touch up on name change...
3358
3359   Revision 1.11  2002/03/26 22:29:55  swa
3360   we have a new homepage!
3361
3362   Revision 1.10  2002/03/24 12:33:01  swa
3363   more additions.
3364
3365   Revision 1.9  2002/03/24 11:01:05  swa
3366   name change
3367
3368   Revision 1.8  2002/03/23 15:13:11  swa
3369   renamed every reference to the old name with foobar.
3370   fixed "application foobar application" tag, fixed
3371   "the foobar" with "foobar". left junkbustser in cvs
3372   comments and remarks to history untouched.
3373
3374   Revision 1.7  2002/03/11 13:13:27  swa
3375   correct feedback channels
3376
3377   Revision 1.6  2002/02/24 14:25:06  jongfoster
3378   Formatting changes.  Now changing the doctype to DocBook XML 4.1
3379   will work - no other changes are needed.
3380
3381   Revision 1.5  2001/10/31 18:16:51  swa
3382   documentation added: howto generate docs in text and html
3383   format, howto move stuff to the webserver.
3384
3385   Revision 1.4  2001/09/23 10:13:48  swa
3386   upload process established. run make webserver and
3387   the documentation is moved to the webserver. documents
3388   are now linked correctly.
3389
3390   Revision 1.3  2001/09/13 15:27:40  swa
3391   cosmetics
3392
3393   Revision 1.2  2001/09/13 15:20:17  swa
3394   merged standards into developer manual
3395
3396   Revision 1.1  2001/09/12 15:36:41  swa
3397   source files for junkbuster documentation
3398
3399   Revision 1.3  2001/09/10 17:43:59  swa
3400   first proposal of a structure.
3401
3402   Revision 1.2  2001/06/13 14:28:31  swa
3403   docs should have an author.
3404
3405   Revision 1.1  2001/06/13 14:20:37  swa
3406   first import of project's documentation for the webserver.
3407
3408   -->
3409
3410 </article>