Fix BE arch issues.

This commit is contained in:
Vít Ondruch 2014-08-15 10:25:37 +02:00
commit b190354d88
2 changed files with 84 additions and 1 deletions

View file

@ -0,0 +1,75 @@
From 953aa33ab33ca6059f85dc8ebb40c3b6bbaae39c Mon Sep 17 00:00:00 2001
From: Greg Hurrell <greg@hurrell.net>
Date: Thu, 14 Aug 2014 07:40:39 -0700
Subject: [PATCH] Fix (spurious) test failure on big-endian architectures
Our example includes a int_64_t value (the PDU's length prefix), which
is endian-sensitive and so needs to be asserted using native endianness.
Fixes:
https://github.com/wincent/Command-T/issues/101
https://bugzilla.redhat.com/show_bug.cgi?id=1130079
---
spec/command-t/watchman/utils_spec.rb | 45 ++++++++++++++++++++++++-----------
1 file changed, 31 insertions(+), 14 deletions(-)
diff --git a/spec/command-t/watchman/utils_spec.rb b/spec/command-t/watchman/utils_spec.rb
index 8f174f7..4bb5c4e 100644
--- a/spec/command-t/watchman/utils_spec.rb
+++ b/spec/command-t/watchman/utils_spec.rb
@@ -379,20 +379,37 @@ def roundtrip(value)
it 'generates a correct serialization' do
# in Ruby 1.8, hashes aren't ordered, so two serializations are possible
- expected = [
- binary(
- "\x00\x01\x06\x49\x00\x00\x00\x00\x00\x00\x00\x00\x03\x03\x02\x03" \
- "\x05query\x02\x03\x0a/some/path\x01\x03\x02\x02\x03\x0aexpression" \
- "\x00\x03\x02\x02\x03\x04type\x02\x03\x01f\x02\x03\x06fields\x00" \
- "\x03\x01\x02\x03\x04name"
- ),
- binary(
- "\x00\x01\x06\x49\x00\x00\x00\x00\x00\x00\x00\x00\x03\x03\x02\x03" \
- "\x05query\x02\x03\x0a/some/path\x01\x03\x02\x02\x03\x06fields\x00" \
- "\x03\x01\x02\x03\x04name\x02\x03\x0aexpression\x00\x03\x02\x02" \
- "\x03\x04type\x02\x03\x01f"
- )
- ]
+ if little_endian?
+ expected = [
+ binary(
+ "\x00\x01\x06\x49\x00\x00\x00\x00\x00\x00\x00\x00\x03\x03\x02\x03" \
+ "\x05query\x02\x03\x0a/some/path\x01\x03\x02\x02\x03\x0a" \
+ "expression\x00\x03\x02\x02\x03\x04type\x02\x03\x01f\x02\x03\x06" \
+ "fields\x00\x03\x01\x02\x03\x04name"
+ ),
+ binary(
+ "\x00\x01\x06\x49\x00\x00\x00\x00\x00\x00\x00\x00\x03\x03\x02\x03" \
+ "\x05query\x02\x03\x0a/some/path\x01\x03\x02\x02\x03\x06fields" \
+ "\x00\x03\x01\x02\x03\x04name\x02\x03\x0aexpression\x00\x03\x02" \
+ "\x02\x03\x04type\x02\x03\x01f"
+ )
+ ]
+ else
+ expected = [
+ binary(
+ "\x00\x01\x06\x00\x00\x00\x00\x00\x00\x00\x49\x00\x03\x03\x02\x03" \
+ "\x05query\x02\x03\x0a/some/path\x01\x03\x02\x02\x03\x0a" \
+ "expression\x00\x03\x02\x02\x03\x04type\x02\x03\x01f\x02\x03\x06" \
+ "fields\x00\x03\x01\x02\x03\x04name"
+ ),
+ binary(
+ "\x00\x01\x06\x00\x00\x00\x00\x00\x00\x00\x49\x00\x03\x03\x02\x03" \
+ "\x05query\x02\x03\x0a/some/path\x01\x03\x02\x02\x03\x06fields" \
+ "\x00\x03\x01\x02\x03\x04name\x02\x03\x0aexpression\x00\x03\x02" \
+ "\x02\x03\x04type\x02\x03\x01f"
+ )
+ ]
+ end
expect(expected).to include(described_class.dump(query))
end
end
--
2.0.3

View file

@ -8,7 +8,7 @@
Name: vim-command-t
Version: 1.10
Release: 2%{?dist}
Release: 3%{?dist}
Summary: An extremely fast, intuitive mechanism for opening files in VIM
Group: Applications/Editors
License: BSD
@ -21,6 +21,9 @@ Source0: command-t-%{version}.tar.gz
# Plug-in AppData for Gnome Software.
# https://github.com/wincent/Command-T/issues/99
Source1: vim-command-t.metainfo.xml
# Fix test failure on BE architectures (rhbz#1130079).
# https://github.com/wincent/Command-T/commit/953aa33ab33ca6059f85dc8ebb40c3b6bbaae39c
Patch0: vim-command-t-1.10-Fix-test-failure-on-big-endian-architectures.patch
Requires: ruby(release)
# Although command-t does not depend on rubygems directly, the RubyGems are
# required by Ruby, but not always (rhbz#845011). So it is necessary to enforce
@ -50,6 +53,8 @@ more weight.
%prep
%setup -q -c
%patch0 -p1
%build
pushd ./ruby/command-t
@ -111,6 +116,9 @@ vim -c ":helptags %{vimfiles}/doc" -c :q &> /dev/null
%changelog
* Fri Aug 15 2014 Vít Ondruch <vondruch@redhat.com> - 1.10-3
- Fix BE arch issues.
* Mon Aug 11 2014 Vít Ondruch <vondruch@redhat.com> - 1.10-2
- Add missing autoload directory.