Merged from trunk -r14525:14548
authorFabian Groffen <grobian@gentoo.org>
Sat, 10 Oct 2009 10:26:14 +0000 (10:26 -0000)
committerFabian Groffen <grobian@gentoo.org>
Sat, 10 Oct 2009 10:26:14 +0000 (10:26 -0000)
   | 14526    | Ask for the output of 'emerge --info' instead of 'emerge    |
   | arfrever | --info =$CATEGORY/$PF'. Suggested by zmedico.               |

   | 14527    | Add back Arfrever's emerge --info change because it's good  |
   | zmedico  | and I was just confused.                                    |

   | 14528    | Handle AUTOCLEAN and NOCOLOR case insensitivity inside      |
   | zmedico  | config.                                                     |

   | 14529    | Add a --color < y | n > option so that color output can be  |
   | zmedico  | forced. This is going to be used by emerge to force color   |
   |          | fetch output when appropriate.                              |

   | 14530    | Force consistent color output, in case we are capturing     |
   | zmedico  | fetch output through a normal pipe due to unavailability of |
   |          | ptys. Thanks to grobian for reporting.                      |

   | 14531    | Bug #274497 - Misc spelling, grammar, and terminology       |
   | zmedico  | fixes, including URL -> URI and FOO's -> FOOs.              |

   | 14532    | Bug #273099 - Add split-log and split-elog FEATURES for     |
   | zmedico  | splitting build logs and elog mod_save logs into category   |
   |          | subdirectories. Thanks to Sebastian Mingramm (few) for this |
   |          | patch.                                                      |

   | 14533    | Add --changed-use as an alias for --reinstall=changed-use.  |
   | zmedico  | Thanks to Jeremy Olexa <darkside@g.o> for the suggestion.   |

   | 14534    | Fix error in --changed-use logic.                           |
   | zmedico  |                                                             |

   | 14535    | Fix ugly 'Candidates' --debug output.                       |
   | zmedico  |                                                             |

   | 14548    | Add news for 2.1.7.                                         |
   | zmedico  |                                                             |

svn path=/main/branches/prefix/; revision=14551

22 files changed:
NEWS
bin/ebuild
bin/misc-functions.sh
cnf/make.conf.amd64-fbsd.diff
cnf/make.conf.amd64.diff
cnf/make.conf.x86-fbsd.diff
cnf/make.conf.x86.diff
doc/qa.docbook
man/ebuild.1
man/ebuild.5
man/emerge.1
man/make.conf.5
man/portage.5
pym/_emerge/EbuildFetcher.py
pym/_emerge/actions.py
pym/_emerge/depgraph.py
pym/_emerge/help.py
pym/_emerge/main.py
pym/portage/__init__.py
pym/portage/dbapi/porttree.py
pym/portage/elog/mod_save.py
pym/portage/getbinpkg.py

diff --git a/NEWS b/NEWS
index d9034e2642c4081ed2fc143ed54e7aab722b09df..42f985d791943bfeef382d46e0d9f8b24869e1cf 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -5,8 +5,12 @@ portage-2.2
 
 * Add link level dependency awareness to emerge --depclean and --prune actions
   in order to protect against uninstallation of required libraries.
-* Add license visibility filtering (GLEP 23)
 * Add support for generic package sets (also see RELEASE-NOTES)
+
+portage-2.1.7
+-------------
+
+* Add license visibility filtering (GLEP 23)
 * Add --accept-properties option for emerge which allows you to use
   --accept-properties=-interactive in order to temporarily exclude
   interactive updates.
index 5493fba8facf9f8195bc1a863e6cdea3c191092b..dee8fe41f9d3f70b4a50857b91aac81dbab43273 100755 (executable)
@@ -40,6 +40,8 @@ force_help = "When used together with the digest or manifest " + \
        "that do not already exist in ${DISTDIR} will be automatically fetched."
 
 parser.add_option("--force", help=force_help, action="store_true", dest="force")
+parser.add_option("--color", help="enable or disable color output",
+       type="choice", choices=("y", "n"))
 parser.add_option("--debug", help="show debug output",
        action="store_true", dest="debug")
 parser.add_option("--ignore-default-opts",
@@ -85,8 +87,14 @@ if debug and "python-trace" in portage.features:
        import portage.debug
        portage.debug.set_trace(True)
 
-if portage.settings["NOCOLOR"] in ("yes","true") or not sys.stdout.isatty():
+if not opts.color == 'y' and \
+       (opts.color == 'n' or \
+       portage.settings.get('NOCOLOR') in ('yes', 'true') or \
+       not sys.stdout.isatty()):
        portage.output.nocolor()
+       portage.settings.unlock()
+       portage.settings['NOCOLOR'] = 'true'
+       portage.settings.lock()
 
 ebuild = pargs.pop(0)
 
index 40ef835d4593689442bba9dd4074973775e5f26e..3ef1d0b21a669f20169a885165731efa3bdd0823 100644 (file)
@@ -68,7 +68,7 @@ install_qa_check() {
                        unset PORTAGE_QUIET
                fi
 
-               # Make sure we disallow insecure RUNPATH/RPATH's
+               # Make sure we disallow insecure RUNPATH/RPATHs
                # Don't want paths that point to the tree where the package was built
                # (older, broken libtools would do this).  Also check for null paths
                # because the loader will search $PWD when it finds null paths.
@@ -80,7 +80,7 @@ install_qa_check() {
                )
                if [[ -n ${f}${x} ]] ; then
                        vecho -ne '\a\n'
-                       eqawarn "QA Notice: The following files contain insecure RUNPATH's"
+                       eqawarn "QA Notice: The following files contain insecure RUNPATHs"
                        eqawarn " Please file a bug about this at http://bugs.gentoo.org/"
                        eqawarn " with the maintaining herd of the package."
                        eqawarn "${f}${f:+${x:+\n}}${x}"
@@ -93,7 +93,7 @@ install_qa_check() {
                        fi
                fi
 
-               # TEXTREL's are baaaaaaaad
+               # TEXTRELs are baaaaaaaad
                # Allow devs to mark things as ignorable ... e.g. things that are
                # binary-only and upstream isn't cooperating (nvidia-glx) ... we
                # allow ebuild authors to set QA_TEXTRELS_arch and QA_TEXTRELS ...
index 1ad4854f6055d4e4c43a772b42c0c5859de8201c..aa07d0657ac862a980dbd14725aa821c221af70b 100644 (file)
@@ -37,7 +37,7 @@
 +# 
 +# CRITICAL WARNINGS: ****************************************************** #
 +# K6 markings are deceptive. Avoid setting -march for them. See Bug #24379. #
-+# Pentium-M CPU's should not enable sse2 until at least gcc-3.4. Bug 50616. #
++# Pentium-M CPUs should not enable sse2 until at least gcc-3.4. Bug 50616. #
 +# GCC 3.3 doesnt support an amd64 specific -march setting, use 3.4.         #
 +# ************************************************************************* #
  #
index 397e8cb362e496e64e58853486cca23743c87773..b4a93fe2a8caf071c0fbbb14b7765d657e0fe5ee 100644 (file)
@@ -37,7 +37,7 @@
 +# 
 +# CRITICAL WARNINGS: ****************************************************** #
 +# K6 markings are deceptive. Avoid setting -march for them. See Bug #24379. #
-+# Pentium-M CPU's should not enable sse2 until at least gcc-3.4. Bug 50616. #
++# Pentium-M CPUs should not enable sse2 until at least gcc-3.4. Bug 50616. #
 +# GCC 3.3 doesnt support an amd64 specific -march setting, use 3.4.         #
 +# ************************************************************************* #
  #
index 1a49edc66cc4326ec7cb50d9b5670bf693e4590e..9fec4f177d234b17f2d85fab798390564deec924 100644 (file)
@@ -43,7 +43,7 @@
 +#
 +# CRITICAL WARNINGS: ****************************************************** #
 +# K6 markings are deceptive. Avoid setting -march for them. See Bug #24379. #
-+# Pentium-M CPU's should not enable sse2 until at least gcc-3.4. Bug 50616. #
++# Pentium-M CPUs should not enable sse2 until at least gcc-3.4. Bug 50616. #
 +# ************************************************************************* #
  #
 +# Decent examples:
index 3e9594abba36847aac146fbf633fc45b8b825468..f90845111309e697d6f6cc3d2ecc94ab3cf2a0c4 100644 (file)
@@ -73,7 +73,7 @@
 +#
 +# CRITICAL WARNINGS: ****************************************************** #
 +# K6 markings are deceptive. Avoid setting -march for them. See Bug #24379. #
-+# Pentium-M CPU's should not enable sse2 until at least gcc-3.4. Bug 50616. #
++# Pentium-M CPUs should not enable sse2 until at least gcc-3.4. Bug 50616. #
 +# ************************************************************************* #
 +#
 +# NOTE: the -On optimization levels are set with the letter O, not -0 (zero).
index e121e6a958200c5a5f1522a5126145effe7b745e..b9ec375eca8ddbba81ff8ccb3de53840729a5f2d 100644 (file)
@@ -17,7 +17,7 @@
   <title>Scanelf: Insecure RUNPATHs</title>
   <para>
    <programlisting>
-    QA Notice: The following files contain insecure RUNPATH's
+    QA Notice: The following files contain insecure RUNPATHs
    </programlisting>
   </para>
   <para>
index 6c134c7cb48404d495057c124e91d1e4a9b404d6..87fd63b79abf812b8cd1a404bf11709cdb603251 100644 (file)
@@ -185,6 +185,11 @@ information is not incorporated into the RPM.
 Run bash with the \-x option, causing it to output verbose debugging
 information to stdout.
 .TP
+.BR "\-\-color < y | n >"
+Enable or disable color output.  This option will override \fINOCOLOR\fR
+(see \fBmake.conf\fR(5)) and may also be used to force color output when stdout
+is not a tty (by default, color is disabled unless stdout is a tty).
+.TP
 .BR "\-\-force"
 When used together with the digest or manifest command,
 this option forces regeneration of
index 56666e730fdfa350a209555f056f961b7a9d3ccb..67f7e39138c6b4aeb6562d03a8cd5c544f74c6c8 100644 (file)
@@ -156,8 +156,8 @@ package will be required to have a version of portage that recognizes
 the EAPI to which this package conforms.
 .TP
 \fBSRC_URI\fR = \fI"http://happy.com/little/${P}.tar.gz"\fR
-Contains a list of URI's for the required source files.  It can contain
-multiple URI's for a single source file.  The list is processed in order
+Contains a list of URIs for the required source files.  It can contain
+multiple URIs for a single source file.  The list is processed in order
 if the file was not found on any of the \fIGENTOO_MIRRORS\fR.
 Beginning with \fBEAPI 2\fR, the output file name of a given URI may be
 customized with a "->" operator on the right hand side, followed by the
@@ -165,7 +165,7 @@ desired output file name. All tokens, including the operator and output
 file name, should be separated by whitespace.
 .TP
 \fBHOMEPAGE\fR = \fI"http://happy.com/"\fR
-Should contain a list of URL's for the sources main sites and other further
+Should contain a list of URIs for the sources main sites and other further
 package dependent information.
 .TP
 \fBKEYWORDS\fR = \fI[\-~][x86,ppc,sparc,mips,alpha,arm,hppa]\fR
@@ -437,7 +437,7 @@ binaries that are not compatible with debugedit.
 files in \fBSRC_URI\fR will not be downloaded from the \fBGENTOO_MIRRORS\fR.
 .TP
 .I primaryuri
-fetch from URL's in \fBSRC_URI\fR before \fBGENTOO_MIRRORS\fR.
+fetch from URIs in \fBSRC_URI\fR before \fBGENTOO_MIRRORS\fR.
 .TP
 .I strip
 final binaries/libraries will not be stripped of debug symbols.
index 63e9879bad42cc7cc564ddac8dc63525c3cbf0ce..23f44286ddcc4e382f6eac02e004cad7873fb984 100644 (file)
@@ -282,6 +282,9 @@ Creates binary packages for all ebuilds processed without actually
 merging the packages.  This comes with the caveat that all build-time 
 dependencies must already be emerged on the system.
 .TP
+.BR "\-\-changed\-use"
+This is an alias for \fB\-\-reinstall=changed\-use\fR.
+.TP
 .BR "\-\-changelog " (\fB\-l\fR)
 Use this in conjunction with the \fB\-\-pretend\fR option.  This will
 show the ChangeLog entries for all the packages that will be upgraded.
index d16f5a1136d555a012c53f6b2ca139a01a53f895..0e7ebf888379aff1aba5b0adfe8adfd7ccf9faef 100644 (file)
@@ -367,6 +367,15 @@ a read-only NFS share. A read-only \fBDISTDIR\fR is not compatible with the
 \fBFEATURES\fR in order to avoid warning messages that are triggered by this
 incompatibility.
 .TP
+split\-elog
+Store logs created by \fBPORTAGE_ELOG_SYSTEM="save"\fR in category
+subdirectories of \fBPORT_LOGDIR/elog\fR, instead of using
+\fBPORT_LOGDIR/elog\fR directly.
+.TP 
+.B split\-log
+Store build logs in category subdirectories of \fBPORT_LOGDIR/build\fR,
+instead of using \fBPORT_LOGDIR\fR directly.
+.TP
 .B splitdebug
 Prior to stripping ELF etdyn and etexec files, the debugging info is 
 stored for later use by various debuggers.  This feature is disabled by
@@ -432,7 +441,7 @@ place\-holders \\${DISTDIR}, \\${FILE} and \\${URI}.  The command should be
 written to place the fetched file at \\${DISTDIR}/\\${FILE}.
 Also see \fBRESUMECOMMAND\fR.
 .TP
-\fBGENTOO_MIRRORS\fR = \fI[URLs]\fR
+\fBGENTOO_MIRRORS\fR = \fI[URIs]\fR
 Insert your space\-seperated list of local mirrors here.  These
 locations are used to download files before the ones listed in
 the \fIebuild scripts\fR. Merging 'mirrorselect' can help.  Entries in this
index 36805f815e3c41a8e785c1cb1398ea4caf9d92b1..50e7e98e58e86412f10ea0cc2db0556445eafd94 100644 (file)
@@ -427,7 +427,7 @@ media\-other
 Contains variables customizing colors. See \fBcolor.map\fR(5).
 .TP
 .BR mirrors
-Whenever portage encounters a mirror:// style URL it will look up the actual 
+Whenever portage encounters a mirror:// style URI it will look up the actual 
 hosts here.  If the mirror set is not found here, it will check the global 
 mirrors file at /usr/portage/profiles/thirdpartymirrors.  You may also set a 
 special mirror type called "local".  This list of mirrors will be checked 
@@ -755,7 +755,7 @@ may contain any of the characters [A\-Za\-z0\-9_\-]. It must not begin with a
 hyphen.
 .TP
 .BR thirdpartymirrors
-Controls the mapping of mirror:// style URLs to actual lists of 
+Controls the mapping of mirror:// style URIs to actual lists of 
 mirrors.  Keeps us from overloading a single server.
 
 .I Format:
index cf759a9314d9baec26153603a2bbc80edb397813..c5d7a8b0ded39a62f0b41e8032cacd1a9724874f 100644 (file)
@@ -68,6 +68,11 @@ class EbuildFetcher(SpawnProcess):
                if debug:
                        fetch_args.append("--debug")
 
+               if not self.background and nocolor not in ('yes', 'true'):
+                       # Force consistent color output, in case we are capturing fetch
+                       # output through a normal pipe due to unavailability of ptys.
+                       fetch_args.append('--color=y')
+
                self.args = fetch_args
                self.env = fetch_env
                SpawnProcess._start(self)
index 671410e163a8e10e2f504d539b9f44c074e9b1f2..6c897a1aba23289af707ad7063b44b3a53f3ff8c 100644 (file)
@@ -2364,14 +2364,6 @@ def action_uninstall(settings, trees, ldpath_mtimes,
 def adjust_config(myopts, settings):
        """Make emerge specific adjustments to the config."""
 
-       # To enhance usability, make some vars case insensitive by forcing them to
-       # lower case.
-       for myvar in ("AUTOCLEAN", "NOCOLOR"):
-               if myvar in settings:
-                       settings[myvar] = settings[myvar].lower()
-                       settings.backup_changes(myvar)
-       del myvar
-
        # Kill noauto as it will break merges otherwise.
        if "noauto" in settings.features:
                settings.features.remove('noauto')
index 5aed665276351593ffcd31c6ec8edacd8826b4c2..1d75f1809850f40f637ee84938f6bc836c3de15e 100644 (file)
@@ -1182,7 +1182,7 @@ class depgraph(object):
                        return 0
 
                if debug:
-                       print("Candidates:", selected_atoms)
+                       print("Candidates:", [str(x) for x in selected_atoms[pkg]])
 
                root_config = self._frozen_config.roots[dep_root]
                vardb = root_config.trees["vartree"].dbapi
@@ -1209,6 +1209,10 @@ class depgraph(object):
                # <=portage-2.1.6.x did.
                for virt_pkg, atoms in selected_atoms.items():
 
+                       if debug:
+                               print("Candidates: %s: %s" % \
+                                       (virt_pkg.cpv, [str(x) for x in atoms]))
+
                        # Just assume depth + 1 here for now, though it's not entirely
                        # accurate since multilple levels of indirect virtual deps may
                        # have been traversed. The _add_pkg call will reset the depth to
index 312952295b432e37444955d03c589d8448e6c544..e2c7a80ea8b8fd090375302457d6f80c88ef8f79 100644 (file)
@@ -291,6 +291,11 @@ def help(myopts, havecolor=1):
                print("              must not exist for the desired package as they cannot be used if")
                print("              they do not exist on the system.")
                print()
+               print("       " + green("--changed-use"))
+               desc = "This is an alias for --reinstall=changed-use."
+               for line in wrap(desc, desc_width):
+                       print(desc_indent + line)
+               print()
                print("       "+green("--changelog")+" ("+green("-l")+" short option)")
                print("              When pretending, also display the ChangeLog entries for packages")
                print("              that will be upgraded.")
index 9f284aecb1338c310491768935f1b9a70c71e173..41c4c02a06435814305bdf92a911848c2c093b8a 100644 (file)
@@ -51,6 +51,7 @@ if sys.hexversion >= 0x3000000:
 options=[
 "--ask",          "--alphabetical",
 "--buildpkg",     "--buildpkgonly",
+"--changed-use",
 "--changelog",    "--columns",
 "--debug",
 "--digest",
@@ -666,6 +667,10 @@ def parse_opts(tmpcmdline, silent=False):
 
        myoptions, myargs = parser.parse_args(args=tmpcmdline)
 
+       if myoptions.changed_use is not False:
+               myoptions.reinstall = "changed-use"
+               myoptions.changed_use = False
+
        if myoptions.deselect == "True":
                myoptions.deselect = True
 
index 7c245f4b6b01ad111515b3104191b4d8afb24c7f..dc88bd823405ddece593943f214931496007c245 100644 (file)
@@ -1492,6 +1492,10 @@ class config(object):
                ('ACCEPT_PROPERTIES',        '*'),
        )
 
+       # To enhance usability, make some vars case insensitive
+       # by forcing them to lower case.
+       _case_insensitive_vars = ('AUTOCLEAN', 'NOCOLOR',)
+
        def __init__(self, clone=None, mycpv=None, config_profile_path=None,
                config_incrementals=None, config_root=None, target_root=None,
                local_config=True, env=None):
@@ -2222,6 +2226,11 @@ class config(object):
 
                        self._init_dirs()
 
+               for k in self._case_insensitive_vars:
+                       if k in self:
+                               self[k] = self[k].lower()
+                               self.backup_changes(k)
+
                if mycpv:
                        self.setcpv(mycpv)
 
@@ -6486,10 +6495,18 @@ def _prepare_workdir(mysettings):
                logid_time = _unicode_decode(time.strftime("%Y%m%d-%H%M%S",
                        time.gmtime(os.stat(logid_path).st_mtime)),
                        encoding=_encodings['content'], errors='replace')
-               mysettings["PORTAGE_LOG_FILE"] = os.path.join(
-                       mysettings["PORT_LOGDIR"], "%s:%s:%s.log" % \
-                       (mysettings["CATEGORY"], mysettings["PF"], logid_time))
-               del logid_path, logid_time
+
+               if "split-log" in mysettings.features:
+                       mysettings["PORTAGE_LOG_FILE"] = os.path.join(
+                               mysettings["PORT_LOGDIR"], "build", "%s/%s:%s.log" % \
+                               (mysettings["CATEGORY"], mysettings["PF"], logid_time))
+               else:
+                       mysettings["PORTAGE_LOG_FILE"] = os.path.join(
+                               mysettings["PORT_LOGDIR"], "%s:%s:%s.log" % \
+                               (mysettings["CATEGORY"], mysettings["PF"], logid_time))
+
+               util.ensure_dirs(os.path.dirname(mysettings["PORTAGE_LOG_FILE"]))
+
        else:
                # NOTE: When sesandbox is enabled, the local SELinux security policies
                # may not allow output to be piped out of the sesandbox domain. The
@@ -6588,7 +6605,7 @@ def doebuild(myebuild, mydo, myroot, mysettings, debug=0, listonly=0,
        @type dbkey: Dict or String
        @param use_cache: Enables the cache
        @type use_cache: Boolean
-       @param fetchall: Used to wrap fetch(), fetches all URI's (even ones invalid due to USE conditionals)
+       @param fetchall: Used to wrap fetch(), fetches all URIs (even ones invalid due to USE conditionals)
        @type fetchall: Boolean
        @param tree: Which tree to use ('vartree','porttree','bintree', etc..), defaults to 'porttree'
        @type tree: String
index ef0b8a6942a3a034223b3fde25aeaf05b90a1def..e955a6a57e970541f944bc8f6cb0b1dece77c971 100644 (file)
@@ -433,7 +433,7 @@ class portdbapi(dbapi):
        def getRepositories(self):
                """
                This function is required for GLEP 42 compliance; it will return a list of
-               repository ID's
+               repository IDs
                TreeMap = {id: path}
                """
                return [k for k in self.treemap if k]
index 44900bb0f0fce29b59ad08f24ec5fffe90543b2e..12d5b71224ff712564388fc13cb8cd00279d67a2 100644 (file)
@@ -22,9 +22,20 @@ def process(mysettings, key, logentries, fulltext):
                elogdir = os.path.join(EPREFIX, "var", "log", "portage", "elog")
        ensure_dirs(elogdir, uid=portage_uid, gid=portage_gid, mode=0o2770)
 
-       elogfilename = os.path.join(elogdir, path + ":" + _unicode_decode(
+       cat = mysettings['CATEGORY']
+       pf = mysettings['PF']
+
+       elogfilename = pf + ":" + _unicode_decode(
                time.strftime("%Y%m%d-%H%M%S", time.gmtime(time.time())),
-               encoding=_encodings['content'], errors='replace') + ".log")
+               encoding=_encodings['content'], errors='replace') + ".log"
+
+       if "split-elog" in mysettings.features:
+               elogfilename = os.path.join(elogdir, cat, elogfilename)
+       else:
+               elogfilename = os.path.join(elogdir, cat + ':' + elogfilename)
+       ensure_dirs(os.path.dirname(elogfilename),
+               uid=portage_uid, gid=portage_gid, mode=0o2770)
+
        elogfile = codecs.open(_unicode_encode(elogfilename,
                encoding=_encodings['fs'], errors='strict'),
                mode='w', encoding=_encodings['content'], errors='backslashreplace')
index 47095afffaee356a841acc2d0f0191cac09adf2a..8fcb0228832e55bef68ab41969a10a1085a6ed9b 100644 (file)
@@ -108,7 +108,7 @@ def create_conn(baseurl,conn=None):
 
        parts = baseurl.split("://",1)
        if len(parts) != 2:
-               raise ValueError(_("Provided URL does not "
+               raise ValueError(_("Provided URI does not "
                        "contain protocol identifier. '%s'") % baseurl)
        protocol,url_parts = parts
        del parts
@@ -316,7 +316,7 @@ def match_in_array(array, prefix="", suffix="", match_both=1, allow_overlap=0):
 
 def dir_get_list(baseurl,conn=None):
        """(baseurl[,connection]) -- Takes a base url to connect to and read from.
-       URL should be in the for <proto>://<site>[:port]<path>
+       URI should be in the form <proto>://<site>[:port]<path>
        Connection is used for persistent connection instances."""
 
        if not conn:
@@ -364,7 +364,7 @@ def dir_get_list(baseurl,conn=None):
 
 def file_get_metadata(baseurl,conn=None, chunk_size=3000):
        """(baseurl[,connection]) -- Takes a base url to connect to and read from.
-       URL should be in the for <proto>://<site>[:port]<path>
+       URI should be in the form <proto>://<site>[:port]<path>
        Connection is used for persistent connection instances."""
 
        if not conn:
@@ -415,7 +415,7 @@ def file_get_metadata(baseurl,conn=None, chunk_size=3000):
 
 def file_get(baseurl,dest,conn=None,fcmd=None):
        """(baseurl,dest,fcmd=) -- Takes a base url to connect to and read from.
-       URL should be in the for <proto>://[user[:pass]@]<site>[:port]<path>"""
+       URI should be in the form <proto>://[user[:pass]@]<site>[:port]<path>"""
 
        if not fcmd:
                return file_get_lib(baseurl,dest,conn)
@@ -443,7 +443,7 @@ def file_get(baseurl,dest,conn=None,fcmd=None):
 
 def file_get_lib(baseurl,dest,conn=None):
        """(baseurl[,connection]) -- Takes a base url to connect to and read from.
-       URL should be in the for <proto>://<site>[:port]<path>
+       URI should be in the form <proto>://<site>[:port]<path>
        Connection is used for persistent connection instances."""
 
        if not conn: