Clojure

broken example in ns docstring

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: Release 1.4
  • Component/s: None
  • Labels:
    None
  • Patch:
    Code
  • Approval:
    Ok

Description

The docstring for `ns` currently has a line

(:require (clojure.contrib sql sql.tests))

which fails with

Java.lang.Exception: lib names inside prefix lists must not contain periods

because of the `sql.tests` value. While the `:use` line will likely also fail,
that one - `(:use (my.lib this that))` - is clearly just an example, whereas the
`:require` line actually shows an incorrect syntax that will never work.

The latest patch swaps `sql.tests` in the docstring for another contrib library, `combinatorics`.

Activity

Hide
Colin Jones added a comment -

This patch is the correct format; using `git format-patch`, instead of dumping the output of `git diff` to a file. Sorry about that.

Show
Colin Jones added a comment - This patch is the correct format; using `git format-patch`, instead of dumping the output of `git diff` to a file. Sorry about that.
Hide
Stuart Sierra added a comment -

Trivial docstring fix. Screened.

Show
Stuart Sierra added a comment - Trivial docstring fix. Screened.

People

Vote (2)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: