Bug 211682

Summary: net/omnitty: Add abililty to "toggle" tagged machines
Product: Ports & Packages Reporter: david
Component: Individual Port(s)Assignee: Tobias Kortkamp <tobik>
Status: Closed FIXED    
Severity: Affects Only Me CC: david, pi, w.schwarzenfeld
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch containing declaration of machmgr_toggle_tag_all() function
none
Patch containing machmgr_toggle_tag_all() function
none
Patch containing code to invoke machmgr_toggle_tag_all() function none

Description david 2016-08-09 02:08:43 UTC
Created attachment 173429 [details]
Patch containing declaration of machmgr_toggle_tag_all() function

I use net/omnitty a fair bit, often with fairly large lists of machines.

I often need to do "something" to about half the machines, then do "something else" to the other half.  So I came up with a hack to toggle the "tagged" status of the machines.

This is implemented by means of the F1 menu; I added the command characters "o" and "O" for this purpose.  (I was going to use "x" and "X", but "X" is already in use.  If someone else has a better/more mnemonic code letters to use... well, I'm in favor!)

Anyway, drop the attached patchfiles into the files/ subdirectory, then make the port as usual.  I went ahead and separated the patchfiles, as I understand that it's traditional to have a separate patchfile for each port file that's being patched, even though they collectively implement a single function.
Comment 1 david 2016-08-09 02:10:25 UTC
Created attachment 173430 [details]
Patch containing machmgr_toggle_tag_all() function

I don't seem to be able to create a PR with multiple attachments. :-/
Comment 2 david 2016-08-09 02:11:13 UTC
Created attachment 173431 [details]
Patch containing code to invoke machmgr_toggle_tag_all() function
Comment 3 commit-hook freebsd_committer freebsd_triage 2018-08-09 12:25:59 UTC
A commit references this bug:

Author: tobik
Date: Thu Aug  9 12:25:09 UTC 2018
New revision: 476742
URL: https://svnweb.freebsd.org/changeset/ports/476742

Log:
  net/omnitty: Add abililty to "toggle" tagged machines

  We often need to do "something" to about half the machines, then
  do "something else" to the other half.  So add a hack to toggle the
  "tagged" status of the machines.

  This is implemented by means of the F1 menu; the command characters
  "o" and "O" can toggle the "tagged" status of the machines.

  PR:		211682
  Submitted by:	maintainer

Changes:
  head/net/omnitty/Makefile
  head/net/omnitty/files/patch-machmgr.c
  head/net/omnitty/files/patch-machmgr.h
  head/net/omnitty/files/patch-main.c
  head/net/omnitty/files/patch-menu.c
Comment 4 Tobias Kortkamp freebsd_committer freebsd_triage 2018-08-09 12:27:10 UTC
Finally committed. Thanks!

This was really annoying to apply.  Having to download and move
individual patch files is no fun.  I did this out of good will now
since the bug is already 2 years old, but please provide a single
diff generated with `svn diff` that we can apply against the port
with `svn patch` in the future.