DaemonForums  

Go Back   DaemonForums > OpenBSD > OpenBSD General

OpenBSD General Other questions regarding OpenBSD which do not fit in any of the categories below.

Reply
 
Thread Tools Display Modes
  #1   (View Single Post)  
Old 2 Weeks Ago
Funkygoby Funkygoby is offline
Port Guard
 
Join Date: Aug 2015
Posts: 22
Default Possible FAQ typos

Hello,
I was wondering where should FAQ corrections be adressed? Via git, on the bug or misc mailing list? Maybe IRC?

What would be the more convenient way for the maintainer?
Reply With Quote
  #2   (View Single Post)  
Old 2 Weeks Ago
jggimi's Avatar
jggimi jggimi is offline
More noise than signal
 
Join Date: May 2008
Location: USA
Posts: 5,670
Default

Checkout a working set of the CVS www repository, and post your proposed unified diffs to tech@.

Last edited by jggimi; 2 Weeks Ago at 12:59 AM. Reason: clarity
Reply With Quote
  #3   (View Single Post)  
Old 2 Weeks Ago
Funkygoby Funkygoby is offline
Port Guard
 
Join Date: Aug 2015
Posts: 22
Default

jggimi, thank you!
So this is what I did after following the instructions here
Code:
cvs -qd anoncvs@anoncvs.ca.openbsd.org:/cvs get -P www/faq
cd www/faq
-- I did my modifications
cvs diff > faq_typos.diff
Here is the patch:
Code:
? faq_typos.diff
Index: faq13.html
===================================================================
RCS file: /cvs/www/faq/faq13.html,v
retrieving revision 1.200
diff -r1.200 faq13.html
462c462
< It shows three MIDI ports, known by
---
> It shows two MIDI ports, known by
Index: ports/index.html
===================================================================
RCS file: /cvs/www/faq/ports/index.html,v
retrieving revision 1.27
diff -r1.27 index.html
124c124
< patch, configure, compile and package the raw soure code.
---
> patch, configure, compile and package the raw source code.
I am going to post this to tech@ hopping that I am doing things right.

Last edited by Funkygoby; 2 Weeks Ago at 12:18 PM. Reason: Typo! Ah the irony
Reply With Quote
  #4   (View Single Post)  
Old 2 Weeks Ago
jggimi's Avatar
jggimi jggimi is offline
More noise than signal
 
Join Date: May 2008
Location: USA
Posts: 5,670
Default

Almost. While your patch might be accepted, please note that OpenBSD uses unified diffs, via the -u option of diff(1).

For our convenience, OpenBSD installs a .cvsrc dot file for all users, which uses -uNp as the default option for cvs(1) diff commands.
Code:
# $OpenBSD: dot.cvsrc,v 1.3 2016/10/31 20:50:11 tb Exp $
#
diff -uNp
update -Pd
checkout -P
rdiff -u
Reply With Quote
  #5   (View Single Post)  
Old 2 Weeks Ago
Funkygoby Funkygoby is offline
Port Guard
 
Join Date: Aug 2015
Posts: 22
Default

Shame on me for using Debian to patch OpenBSD...
Here is the complete story, first I used diff(1) with the -u option as you suggested then I saw the cvs(1) diff command and assumed that it would work the same on OpenBSD and Debian and wouldn't need additional options.

Anyway, the patch applied "cleanly" and was accepted. The dev was kind enough to point my mistakes.

Lesson learned.
Reply With Quote
Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT. The time now is 12:58 PM.


Powered by vBulletin® Version 3.8.4
Copyright ©2000 - 2017, Jelsoft Enterprises Ltd.
Content copyright © 2007-2010, the authors
Daemon image copyright ©1988, Marshall Kirk McKusick