Sync ragel 6.6-1 (universe) from Debian unstable (main)

Bug #589881 reported by Felix Geyer on 2010-06-04
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ragel (Ubuntu)

Bug Description

Please sync ragel 6.6-1 (universe) from Debian unstable (main)

Explanation of the Ubuntu delta and why it can be dropped:
Ubuntu delta:
> * ragel/main.cpp: Use and check strdup(), fixes FTBFS
and a follow-up fix:
> * ragel/main.cpp: Fix mem leak

This has been fixed upstream in 6.6 (checked the source code and test-built on maverick).
Upstream changelog mentions this as well:
> -Applied a number of patches from Diego Elio 'Flameeyes' Pettenò. Should not
> be modifying the program's arguments. Problem raised by const correctness in
> gcc 4.4. Other const-correctness and include fixes provided.

Changelog entries since current maverick version 6.5-1ubuntu2:

ragel (6.6-1) unstable; urgency=low

  * New upstream release

 -- Robert Lemmen <email address hidden> Tue, 05 Jan 2010 15:16:17 +0000

Benjamin Drung (bdrung) wrote :

Two function were patched: processArgs and makeIntermedTemplate. Upstream fixed processArgs, but didn't changed makeIntermedTemplate. Therefore I think that we can't drop the patch part for makeIntermedTemplate and should forward this to upstream.

Please resubscribe ubuntu-sponsors once you have addressed this point.

Felix Geyer (debfx) wrote :

Upstream fixed makeIntermedTemplate().
Instead of using strdup() the variable type has been changed to const char*.

@@ -507,7 +514,7 @@
  char *result = 0;
  const char *templ = "ragel-XXXXXX.xml";
- char *lastSlash = strrchr( baseFileName, '/' );
+ const char *lastSlash = strrchr( baseFileName, '/' );
  if ( lastSlash == 0 ) {
   result = new char[strlen(templ)+1];
   strcpy( result, templ );

Benjamin Drung (bdrung) on 2010-06-21
Changed in ragel (Ubuntu):
importance: Undecided → Wishlist
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ragel - 6.6-1

ragel (6.6-1) unstable; urgency=low

  * New upstream release
 -- Felix Geyer <email address hidden> Tue, 05 Jan 2010 15:16:17 +0000

Changed in ragel (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers