Your message dated Sun, 30 Dec 2012 20:15:07 +0100
with message-id <20121230191507.gx5...@radis.cristau.org>
and subject line Re: Bug#696456: unblock: vite/1.2+svn1347-3
has caused the Debian Bug report #696456,
regarding unblock: vite/1.2+svn1347-3
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)
--
696456: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=696456
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian....@packages.debian.org
Usertags: unblock
Please unblock package vite
unblock vite/1.2+svn1347-3
This is a non-intrusive port of an upstream fix for the paje trace
format (one of the three formats supported by vite).
The Paje tools have indeed changed the way they output files since some
time already (while still respecting the established standard), making
it completely unreadable with vite because vite was not ready for such
interpretation of the standard. That went unnoticed until now because
the vite maintainers use their trunk which had the fix already.
Samuel
diff -Nru vite-1.2+svn1347/debian/changelog vite-1.2+svn1347/debian/changelog
--- vite-1.2+svn1347/debian/changelog 2012-08-03 22:39:39.000000000 +0200
+++ vite-1.2+svn1347/debian/changelog 2012-12-11 17:41:35.000000000 +0100
@@ -1,3 +1,9 @@
+vite (1.2+svn1347-3) unstable; urgency=low
+
+ * patches/paje: New patch to fix paje parsing (Closes: Bug#695678).
+
+ -- Samuel Thibault <sthiba...@debian.org> Tue, 11 Dec 2012 17:04:46 +0100
+
vite (1.2+svn1347-2) unstable; urgency=low
* patches/overflow.patch: New upstream patch to fix overflow on big input
diff -Nru vite-1.2+svn1347/debian/patches/paje
vite-1.2+svn1347/debian/patches/paje
--- vite-1.2+svn1347/debian/patches/paje 1970-01-01 01:00:00.000000000
+0100
+++ vite-1.2+svn1347/debian/patches/paje 2012-12-11 17:19:33.000000000
+0100
@@ -0,0 +1,168 @@
+--- a/src/parser/PajeParser/PajeDefinition.hpp
++++ b/src/parser/PajeParser/PajeDefinition.hpp
+@@ -82,23 +82,28 @@
+ * the need of the user.
+ */
+ #define FIELDNAME_SIZEMAX 32
+-#define FIELDNAME_SIZE 16
++#define FIELDNAME_SIZE 14
+ #define _PajeFN_Time 0
+ #define _PajeFN_Name 1
+ #define _PajeFN_Alias 2
+-#define _PajeFN_ContainerType 3
+-#define _PajeFN_EntityType 4
+-#define _PajeFN_Type 5
+-#define _PajeFN_Container 6
+-#define _PajeFN_SourceContainerType 7
+-#define _PajeFN_DestContainerType 8
+-#define _PajeFN_SourceContainer 9
+-#define _PajeFN_DestContainer 10
+-#define _PajeFN_Color 11
+-#define _PajeFN_Value 12
+-#define _PajeFN_Key 13
+-#define _PajeFN_File 14
+-#define _PajeFN_Line 15
++#define _PajeFN_Type 3
++#define _PajeFN_Container 4
++#define _PajeFN_SourceContainerType 5
++#define _PajeFN_DestContainerType 6
++#define _PajeFN_SourceContainer 7
++#define _PajeFN_DestContainer 8
++#define _PajeFN_Color 9
++#define _PajeFN_Value 10
++#define _PajeFN_Key 11
++#define _PajeFN_File 12
++#define _PajeFN_Line 13
++
++#define _PajeFN_ContainerType 3
++#define _PajeFN_EntityType 3
++#define _PajeFN_StartContainerType 5
++#define _PajeFN_EndContainerType 6
++#define _PajeFN_StartContainer 7
++#define _PajeFN_EndContainer 8
+
+ /*!
+ * \brief Types for the fields
+--- a/src/parser/PajeParser/ParserDefinitionPaje.cpp
++++ b/src/parser/PajeParser/ParserDefinitionPaje.cpp
+@@ -135,8 +135,6 @@ ParserDefinitionPaje::ParserDefinitionPa
+ INSERT_FNAME(Time, _FieldType_Date );
+ INSERT_FNAME(Name, _FieldType_Int | _FieldType_String );
+ INSERT_FNAME(Alias, _FieldType_Int | _FieldType_String );
+- INSERT_FNAME(ContainerType, _FieldType_Int | _FieldType_String );
+- INSERT_FNAME(EntityType, _FieldType_Int | _FieldType_String );
+ INSERT_FNAME(Type, _FieldType_Int | _FieldType_String );
+ INSERT_FNAME(Container, _FieldType_Int | _FieldType_String );
+ INSERT_FNAME(SourceContainerType, _FieldType_Int | _FieldType_String );
+@@ -154,7 +152,14 @@ ParserDefinitionPaje::ParserDefinitionPa
+ for(i=0; i<FIELDNAME_SIZE; i++) {
+ _FNameStr2Id.insert(pair<string, int>(_FieldNames[i]._name, i));
+ }
+-
++
++ _FNameStr2Id.insert(pair<string, int>("ContainerType" ,
_PajeFN_ContainerType ));
++ _FNameStr2Id.insert(pair<string, int>("EntityType" ,
_PajeFN_EntityType ));
++ _FNameStr2Id.insert(pair<string, int>("StartContainerType",
_PajeFN_StartContainerType ));
++ _FNameStr2Id.insert(pair<string, int>("EndContainerType" ,
_PajeFN_EndContainerType ));
++ _FNameStr2Id.insert(pair<string, int>("StartContainer" ,
_PajeFN_StartContainer ));
++ _FNameStr2Id.insert(pair<string, int>("EndContainer" ,
_PajeFN_EndContainer ));
++
+ // Initialize the list of Events available
+ i = 0;
+ _EventDefs.resize(PAJE_EVENTDEF_SIZE);
+--- a/src/parser/PajeParser/ParserEventPaje.cpp
++++ b/src/parser/PajeParser/ParserEventPaje.cpp
+@@ -80,12 +80,12 @@ void ParserEventPaje::store_event(const
+ string fvalue;
+ string alias;
+ string name;
+- String container_type;
+ String source_container_type;
+ String dest_container_type;
+- String entity_type;
+ Date time;
+ String type;
++#define container_type type
++#define entity_type type
+ String container;
+ String value_string;
+ Double value_double;
+@@ -150,9 +150,6 @@ void ParserEventPaje::store_event(const
+ name = fvalue;
+ break;
+
+- case _PajeFN_ContainerType :
+- container_type = fvalue;
+- break;
+
+ case _PajeFN_SourceContainerType :
+ source_container_type = fvalue;
+@@ -162,10 +159,6 @@ void ParserEventPaje::store_event(const
+ dest_container_type = fvalue;
+ break;
+
+- case _PajeFN_EntityType :
+- entity_type = fvalue;
+- break;
+-
+ case _PajeFN_Time :
+ time = fvalue;
+ if(!time.is_correct()) {
+@@ -177,6 +170,8 @@ void ParserEventPaje::store_event(const
+ }
+ break;
+
++ //case _PajeFN_ContainerType :
++ //case _PajeFN_EntityType :
+ case _PajeFN_Type :
+ type = fvalue;
+ break;
+--- a/src/trace/TraceBuilderThread.hpp
++++ b/src/trace/TraceBuilderThread.hpp
+@@ -41,11 +41,11 @@ typedef struct Trace_builder_struct{
+ void (*func)(Trace_builder_struct*);
+ Date time;
+ Name alias;
+- String container_type;
+ String source_container_type;
+ String dest_container_type;
+- String entity_type;
+ String type;
++#define container_type type
++#define entity_type type
+ String container;
+ String value_string;
+ Double value_double;
+--- a/src/parser/PajeParser/mt_ParserEventPaje.cpp
++++ b/src/parser/PajeParser/mt_ParserEventPaje.cpp
+@@ -146,10 +146,6 @@ int mt_ParserEventPaje::store_event(cons
+ name = fvalue;
+ break;
+
+- case _PajeFN_ContainerType :
+- tb_struct->container_type = fvalue;
+- break;
+-
+ case _PajeFN_SourceContainerType :
+ tb_struct->source_container_type = fvalue;
+ break;
+@@ -158,10 +154,6 @@ int mt_ParserEventPaje::store_event(cons
+ tb_struct->dest_container_type = fvalue;
+ break;
+
+- case _PajeFN_EntityType :
+- tb_struct->entity_type = fvalue;
+- break;
+-
+ case _PajeFN_Time :
+ tb_struct->time = fvalue;
+ if(!tb_struct->time.is_correct()) {
+@@ -173,6 +165,8 @@ int mt_ParserEventPaje::store_event(cons
+ }
+ break;
+
++ //case _PajeFN_ContainerType :
++ //case _PajeFN_EntityType :
+ case _PajeFN_Type :
+ tb_struct->type = fvalue;
+ break;
diff -Nru vite-1.2+svn1347/debian/patches/series
vite-1.2+svn1347/debian/patches/series
--- vite-1.2+svn1347/debian/patches/series 2012-08-03 22:39:39.000000000
+0200
+++ vite-1.2+svn1347/debian/patches/series 2012-12-11 16:49:31.000000000
+0100
@@ -1 +1,2 @@
overflow.patch
+paje
--- End Message ---
--- Begin Message ---
On Sun, Dec 30, 2012 at 19:54:29 +0100, intrigeri wrote:
> Hi,
>
> (disclaimer: I'm not part of the release team, just trying to help
> a bit with low hanging fruits.)
>
> Samuel Thibault wrote (20 Dec 2012 22:41:08 GMT) :
> > unblock vite/1.2+svn1347-3
>
> > This is a non-intrusive port of an upstream fix for the paje trace
> > format (one of the three formats supported by vite).
>
> Thanks.
>
> No reverse dependencies, priority: optional, leaf package.
>
> I don't feel I'm qualified to review the actual code changes,
> but they are indeed non-intrusive, relatively short,
> and to the point as far as I can tell.
>
Unblocked.
Cheers,
Julien
signature.asc
Description: Digital signature
--- End Message ---
