Bug 194910 - [PATCH] Mention svnlite(1) in Makefile.inc1, make.conf(5) and build(7)
Summary: [PATCH] Mention svnlite(1) in Makefile.inc1, make.conf(5) and build(7)
Status: Closed FIXED
Alias: None
Product: Documentation
Classification: Unclassified
Component: Manual Pages (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-bugs (Nobody)
Depends on:
Reported: 2014-11-09 14:49 UTC by Juan Ramón Molina Menor
Modified: 2018-08-11 13:20 UTC (History)
2 users (show)

See Also:

Patch for adding svnlite(1) to Makefile.inc1, make.conf(5) and build(7) (1.35 KB, patch)
2014-11-09 14:49 UTC, Juan Ramón Molina Menor
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Juan Ramón Molina Menor 2014-11-09 14:49:43 UTC
Created attachment 149222 [details]
Patch for adding svnlite(1) to Makefile.inc1, make.conf(5) and build(7)

There seems to be an inconsistency with svnlite(1). The initial commit (r251886) says that it was included for checking out and committing source. Nevertheless, the make.conf man page says, in the SVN_UPDATE section, that no subversion client is included in the base system, and indeed 'make update' does not work by default, as it searches for /usr/local/bin/svn.

I have attached a naive patch which replaces svn(1) by svnlite(1) in the build(7) and make.conf(5) pages, as well as the path to svnlite(1) in Makefile.inc1.

In Makefile.inc1, it would be nice to account for the possibility of an user setting WITH_SVN in src.conf(5) and thus obtaining a 'svn' binary instead of 'svnlite', but I'm not competent enough to do this properly.

If the idea of this patch is accepted, I think we should also consider resolving #194080.

Hope it helps,
Comment 1 commit-hook freebsd_committer 2018-08-11 13:19:22 UTC
A commit references this bug:

Author: netchild
Date: Sat Aug 11 13:18:20 UTC 2018
New revision: 337621
URL: https://svnweb.freebsd.org/changeset/base/337621

  Add svnlite to places where svn is mentioned.

  The Makefile part in the PR is solved already differently, so this
  part is skipped form the PR The man page change change is slightly
  changed to adapt to the way the Makefile works and to the spirit
  of what is intended here.

  Submitted by:	Juan Ram?n Molina Menor <info@juanmolina.eu>
  PR:		194910
  Sponsored by:	Essen Hackathon

Comment 2 Alexander Leidinger freebsd_committer 2018-08-11 13:20:46 UTC
Committed (except Makefile, as already solved differently), and slight rework of the man page to match what the Makefile does.