Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] mosh: Fix patch
@ 2019-11-11  7:36 voidlinux-github
  2019-11-11 21:42 ` [PR PATCH] [Updated] " voidlinux-github
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: voidlinux-github @ 2019-11-11  7:36 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 408 bytes --]

There is a new pull request by ndowens against master on the void-packages repository

https://github.com/ndowens/void-packages mosh
https://github.com/void-linux/void-packages/pull/16357

mosh: Fix patch
assert(false); in patch causes the fish shell
to fail to start, essentially causing lock in
a mobile terminal

A patch file from https://github.com/void-linux/void-packages/pull/16357.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mosh-16357.patch --]
[-- Type: text/x-diff, Size: 1429 bytes --]

From 79a996fcf49619954cfc721f117391719ed9b8d6 Mon Sep 17 00:00:00 2001
From: Nathan Owens <ndowens04@gmail.com>
Date: Mon, 11 Nov 2019 01:32:14 -0600
Subject: [PATCH] mosh: Fix patch

assert(false); in patch causes the fish shell
to fail to start, essentially causing lock in
a mobile terminal
---
 srcpkgs/mosh/patches/ignore-unknown-renditions.patch | 2 +-
 srcpkgs/mosh/template                                | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/mosh/patches/ignore-unknown-renditions.patch b/srcpkgs/mosh/patches/ignore-unknown-renditions.patch
index c1c805b2b23..acd667a7cde 100644
--- a/srcpkgs/mosh/patches/ignore-unknown-renditions.patch
+++ b/srcpkgs/mosh/patches/ignore-unknown-renditions.patch
@@ -41,7 +41,7 @@ index a724afa6..93a31553 100644
    case 5: case 25: set_attribute(blink, value); break;
    case 7: case 27: set_attribute(inverse, value); break;
    case 8: case 28: set_attribute(invisible, value); break;
-+  default: assert(false);
++default: break;
    }
  }
  
diff --git a/srcpkgs/mosh/template b/srcpkgs/mosh/template
index bb333076cba..ac3d8112a79 100644
--- a/srcpkgs/mosh/template
+++ b/srcpkgs/mosh/template
@@ -1,7 +1,7 @@
 # Template file for 'mosh'
 pkgname=mosh
 version=1.3.2
-revision=13
+revision=14
 build_style=gnu-configure
 hostmakedepends="pkg-config protobuf"
 makedepends="ncurses-devel protobuf-devel libutempter-devel libressl-devel"

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PR PATCH] [Updated] mosh: Fix patch
  2019-11-11  7:36 [PR PATCH] mosh: Fix patch voidlinux-github
@ 2019-11-11 21:42 ` voidlinux-github
  2019-11-11 21:44 ` voidlinux-github
  2019-11-12 11:59 ` [PR PATCH] [Merged]: " voidlinux-github
  2 siblings, 0 replies; 4+ messages in thread
From: voidlinux-github @ 2019-11-11 21:42 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 413 bytes --]

There is an updated pull request by ndowens against master on the void-packages repository

https://github.com/ndowens/void-packages mosh
https://github.com/void-linux/void-packages/pull/16357

mosh: Fix patch
assert(false); in patch causes the fish shell
to fail to start, essentially causing lock in
a mobile terminal

A patch file from https://github.com/void-linux/void-packages/pull/16357.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mosh-16357.patch --]
[-- Type: text/x-diff, Size: 4879 bytes --]

From a6c6e0f3bcec177626dee77374313041b6278d27 Mon Sep 17 00:00:00 2001
From: Nathan Owens <ndowens04@gmail.com>
Date: Mon, 11 Nov 2019 01:32:14 -0600
Subject: [PATCH] mosh: Remove patch

Patch causes issue with users using fish shell
---
 .../patches/ignore-unknown-renditions.patch   | 122 ------------------
 srcpkgs/mosh/template                         |   2 +-
 2 files changed, 1 insertion(+), 123 deletions(-)
 delete mode 100644 srcpkgs/mosh/patches/ignore-unknown-renditions.patch

diff --git a/srcpkgs/mosh/patches/ignore-unknown-renditions.patch b/srcpkgs/mosh/patches/ignore-unknown-renditions.patch
deleted file mode 100644
index c1c805b2b23..00000000000
--- a/srcpkgs/mosh/patches/ignore-unknown-renditions.patch
+++ /dev/null
@@ -1,122 +0,0 @@
-From 4835dcf5eed93f8dee44e5d1932c38b1cd243d41 Mon Sep 17 00:00:00 2001
-From: John Hood <cgull@glup.org>
-Date: Mon, 7 Nov 2016 01:00:03 -0500
-Subject: [PATCH] Various switch statement fixes.
-
----
- src/frontend/mosh-server.cc         |  5 +++--
- src/terminal/terminalframebuffer.cc |  1 +
- src/terminal/terminalfunctions.cc   | 22 ++++++++++++++++------
- 3 files changed, 20 insertions(+), 8 deletions(-)
-
-diff --git src/frontend/mosh-server.cc src/frontend/mosh-server.cc
-index 386fcf92..71acc74d 100644
---- src/frontend/mosh-server.cc
-+++ src/frontend/mosh-server.cc
-@@ -250,8 +250,9 @@ int main( int argc, char *argv[] )
- 	locale_vars.push_back( string( optarg ) );
- 	break;
-       default:
--	print_usage( stderr, argv[ 0 ] );
- 	/* don't die on unknown options */
-+	print_usage( stderr, argv[ 0 ] );
-+	break;
-       }
-     }
-   } else if ( argc == 1 ) {
-@@ -658,7 +659,7 @@ static void serve( int host_fd, Terminal::Complete &terminal, ServerConnection &
- 
-   bool child_released = false;
- 
--  while ( 1 ) {
-+  while ( true ) {
-     try {
-       static const uint64_t timeout_if_no_client = 60000;
-       int timeout = INT_MAX;
-diff --git src/terminal/terminalframebuffer.cc src/terminal/terminalframebuffer.cc
-index a724afa6..93a31553 100644
---- src/terminal/terminalframebuffer.cc
-+++ src/terminal/terminalframebuffer.cc
-@@ -512,6 +512,7 @@ void Renditions::set_rendition( color_type num )
-   case 5: case 25: set_attribute(blink, value); break;
-   case 7: case 27: set_attribute(inverse, value); break;
-   case 8: case 28: set_attribute(invisible, value); break;
-+  default: assert(false);
-   }
- }
- 
-diff --git src/terminal/terminalfunctions.cc src/terminal/terminalfunctions.cc
-index 87cb95c4..a0210e34 100644
---- src/terminal/terminalfunctions.cc
-+++ src/terminal/terminalfunctions.cc
-@@ -64,6 +64,8 @@ static void CSI_EL( Framebuffer *fb, Dispatcher *dispatch )
-   case 2: /* all of line */
-     fb->reset_row( fb->get_mutable_row( -1 ) );
-     break;
-+  default:
-+    break;
-   }
- }
- 
-@@ -89,6 +91,8 @@ static void CSI_ED( Framebuffer *fb, Dispatcher *dispatch ) {
-       fb->reset_row( fb->get_mutable_row( y ) );
-     }
-     break;
-+  default:
-+    break;
-   }
- }
- 
-@@ -114,10 +118,11 @@ static void CSI_cursormove( Framebuffer *fb, Dispatcher *dispatch )
-     break;
-   case 'H':
-   case 'f':
--    int x = dispatch->getparam( 0, 1 );
--    int y = dispatch->getparam( 1, 1 );
--    fb->ds.move_row( x - 1 );
--    fb->ds.move_col( y - 1 );
-+    fb->ds.move_row( dispatch->getparam( 0, 1 ) - 1 );
-+    fb->ds.move_col( dispatch->getparam( 1, 1 ) - 1 );
-+    break;
-+  default:
-+    break;
-   }
- }
- 
-@@ -261,6 +266,8 @@ static void CSI_TBC( Framebuffer *fb, Dispatcher *dispatch )
-       fb->ds.clear_tab( x );
-     }
-     break;
-+  default:
-+    break;
-   }
- }
- 
-@@ -295,6 +302,8 @@ static bool *get_DEC_mode( int param, Framebuffer *fb ) {
-     return &(fb->ds.mouse_alternate_scroll);
-   case 2004: /* bracketed paste */
-     return &(fb->ds.bracketed_paste);
-+  default:
-+    break;
-   }
-   return NULL;
- }
-@@ -340,8 +349,7 @@ static Function func_CSI_DECSM( CSI, "?h", CSI_DECSM, false );
- static Function func_CSI_DECRM( CSI, "?l", CSI_DECRM, false );
- 
- static bool *get_ANSI_mode( int param, Framebuffer *fb ) {
--  switch ( param ) {
--  case 4: /* insert/replace mode */
-+  if ( param == 4 ) { /* insert/replace mode */
-     return &(fb->ds.insert_mode);
-   }
-   return NULL;
-@@ -452,6 +460,8 @@ static void CSI_DSR( Framebuffer *fb, Dispatcher *dispatch )
- 	      fb->ds.get_cursor_col() + 1 );
-     dispatch->terminal_to_host.append( cpr );
-     break;
-+  default:
-+    break;
-   }
- }
- 
diff --git a/srcpkgs/mosh/template b/srcpkgs/mosh/template
index bb333076cba..ac3d8112a79 100644
--- a/srcpkgs/mosh/template
+++ b/srcpkgs/mosh/template
@@ -1,7 +1,7 @@
 # Template file for 'mosh'
 pkgname=mosh
 version=1.3.2
-revision=13
+revision=14
 build_style=gnu-configure
 hostmakedepends="pkg-config protobuf"
 makedepends="ncurses-devel protobuf-devel libutempter-devel libressl-devel"

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PR PATCH] [Updated] mosh: Fix patch
  2019-11-11  7:36 [PR PATCH] mosh: Fix patch voidlinux-github
  2019-11-11 21:42 ` [PR PATCH] [Updated] " voidlinux-github
@ 2019-11-11 21:44 ` voidlinux-github
  2019-11-12 11:59 ` [PR PATCH] [Merged]: " voidlinux-github
  2 siblings, 0 replies; 4+ messages in thread
From: voidlinux-github @ 2019-11-11 21:44 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 413 bytes --]

There is an updated pull request by ndowens against master on the void-packages repository

https://github.com/ndowens/void-packages mosh
https://github.com/void-linux/void-packages/pull/16357

mosh: Fix patch
assert(false); in patch causes the fish shell
to fail to start, essentially causing lock in
a mobile terminal

A patch file from https://github.com/void-linux/void-packages/pull/16357.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mosh-16357.patch --]
[-- Type: text/x-diff, Size: 4926 bytes --]

From db8dced56c54ee41cca97bc56386a50671dd7466 Mon Sep 17 00:00:00 2001
From: Nathan Owens <ndowens04@gmail.com>
Date: Mon, 11 Nov 2019 01:32:14 -0600
Subject: [PATCH] mosh: Remove patch

Patch causes issue with users using fish shell
Was also reverted in mobile-shell/mosh@a40b676
---
 .../patches/ignore-unknown-renditions.patch   | 122 ------------------
 srcpkgs/mosh/template                         |   2 +-
 2 files changed, 1 insertion(+), 123 deletions(-)
 delete mode 100644 srcpkgs/mosh/patches/ignore-unknown-renditions.patch

diff --git a/srcpkgs/mosh/patches/ignore-unknown-renditions.patch b/srcpkgs/mosh/patches/ignore-unknown-renditions.patch
deleted file mode 100644
index c1c805b2b23..00000000000
--- a/srcpkgs/mosh/patches/ignore-unknown-renditions.patch
+++ /dev/null
@@ -1,122 +0,0 @@
-From 4835dcf5eed93f8dee44e5d1932c38b1cd243d41 Mon Sep 17 00:00:00 2001
-From: John Hood <cgull@glup.org>
-Date: Mon, 7 Nov 2016 01:00:03 -0500
-Subject: [PATCH] Various switch statement fixes.
-
----
- src/frontend/mosh-server.cc         |  5 +++--
- src/terminal/terminalframebuffer.cc |  1 +
- src/terminal/terminalfunctions.cc   | 22 ++++++++++++++++------
- 3 files changed, 20 insertions(+), 8 deletions(-)
-
-diff --git src/frontend/mosh-server.cc src/frontend/mosh-server.cc
-index 386fcf92..71acc74d 100644
---- src/frontend/mosh-server.cc
-+++ src/frontend/mosh-server.cc
-@@ -250,8 +250,9 @@ int main( int argc, char *argv[] )
- 	locale_vars.push_back( string( optarg ) );
- 	break;
-       default:
--	print_usage( stderr, argv[ 0 ] );
- 	/* don't die on unknown options */
-+	print_usage( stderr, argv[ 0 ] );
-+	break;
-       }
-     }
-   } else if ( argc == 1 ) {
-@@ -658,7 +659,7 @@ static void serve( int host_fd, Terminal::Complete &terminal, ServerConnection &
- 
-   bool child_released = false;
- 
--  while ( 1 ) {
-+  while ( true ) {
-     try {
-       static const uint64_t timeout_if_no_client = 60000;
-       int timeout = INT_MAX;
-diff --git src/terminal/terminalframebuffer.cc src/terminal/terminalframebuffer.cc
-index a724afa6..93a31553 100644
---- src/terminal/terminalframebuffer.cc
-+++ src/terminal/terminalframebuffer.cc
-@@ -512,6 +512,7 @@ void Renditions::set_rendition( color_type num )
-   case 5: case 25: set_attribute(blink, value); break;
-   case 7: case 27: set_attribute(inverse, value); break;
-   case 8: case 28: set_attribute(invisible, value); break;
-+  default: assert(false);
-   }
- }
- 
-diff --git src/terminal/terminalfunctions.cc src/terminal/terminalfunctions.cc
-index 87cb95c4..a0210e34 100644
---- src/terminal/terminalfunctions.cc
-+++ src/terminal/terminalfunctions.cc
-@@ -64,6 +64,8 @@ static void CSI_EL( Framebuffer *fb, Dispatcher *dispatch )
-   case 2: /* all of line */
-     fb->reset_row( fb->get_mutable_row( -1 ) );
-     break;
-+  default:
-+    break;
-   }
- }
- 
-@@ -89,6 +91,8 @@ static void CSI_ED( Framebuffer *fb, Dispatcher *dispatch ) {
-       fb->reset_row( fb->get_mutable_row( y ) );
-     }
-     break;
-+  default:
-+    break;
-   }
- }
- 
-@@ -114,10 +118,11 @@ static void CSI_cursormove( Framebuffer *fb, Dispatcher *dispatch )
-     break;
-   case 'H':
-   case 'f':
--    int x = dispatch->getparam( 0, 1 );
--    int y = dispatch->getparam( 1, 1 );
--    fb->ds.move_row( x - 1 );
--    fb->ds.move_col( y - 1 );
-+    fb->ds.move_row( dispatch->getparam( 0, 1 ) - 1 );
-+    fb->ds.move_col( dispatch->getparam( 1, 1 ) - 1 );
-+    break;
-+  default:
-+    break;
-   }
- }
- 
-@@ -261,6 +266,8 @@ static void CSI_TBC( Framebuffer *fb, Dispatcher *dispatch )
-       fb->ds.clear_tab( x );
-     }
-     break;
-+  default:
-+    break;
-   }
- }
- 
-@@ -295,6 +302,8 @@ static bool *get_DEC_mode( int param, Framebuffer *fb ) {
-     return &(fb->ds.mouse_alternate_scroll);
-   case 2004: /* bracketed paste */
-     return &(fb->ds.bracketed_paste);
-+  default:
-+    break;
-   }
-   return NULL;
- }
-@@ -340,8 +349,7 @@ static Function func_CSI_DECSM( CSI, "?h", CSI_DECSM, false );
- static Function func_CSI_DECRM( CSI, "?l", CSI_DECRM, false );
- 
- static bool *get_ANSI_mode( int param, Framebuffer *fb ) {
--  switch ( param ) {
--  case 4: /* insert/replace mode */
-+  if ( param == 4 ) { /* insert/replace mode */
-     return &(fb->ds.insert_mode);
-   }
-   return NULL;
-@@ -452,6 +460,8 @@ static void CSI_DSR( Framebuffer *fb, Dispatcher *dispatch )
- 	      fb->ds.get_cursor_col() + 1 );
-     dispatch->terminal_to_host.append( cpr );
-     break;
-+  default:
-+    break;
-   }
- }
- 
diff --git a/srcpkgs/mosh/template b/srcpkgs/mosh/template
index bb333076cba..ac3d8112a79 100644
--- a/srcpkgs/mosh/template
+++ b/srcpkgs/mosh/template
@@ -1,7 +1,7 @@
 # Template file for 'mosh'
 pkgname=mosh
 version=1.3.2
-revision=13
+revision=14
 build_style=gnu-configure
 hostmakedepends="pkg-config protobuf"
 makedepends="ncurses-devel protobuf-devel libutempter-devel libressl-devel"

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PR PATCH] [Merged]: mosh: Fix patch
  2019-11-11  7:36 [PR PATCH] mosh: Fix patch voidlinux-github
  2019-11-11 21:42 ` [PR PATCH] [Updated] " voidlinux-github
  2019-11-11 21:44 ` voidlinux-github
@ 2019-11-12 11:59 ` voidlinux-github
  2 siblings, 0 replies; 4+ messages in thread
From: voidlinux-github @ 2019-11-12 11:59 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 260 bytes --]

There's a merged pull request on the void-packages repository

mosh: Fix patch
https://github.com/void-linux/void-packages/pull/16357

Description:
assert(false); in patch causes the fish shell
to fail to start, essentially causing lock in
a mobile terminal

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-11-12 11:59 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-11  7:36 [PR PATCH] mosh: Fix patch voidlinux-github
2019-11-11 21:42 ` [PR PATCH] [Updated] " voidlinux-github
2019-11-11 21:44 ` voidlinux-github
2019-11-12 11:59 ` [PR PATCH] [Merged]: " voidlinux-github

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).