Bug 183778

Summary: [patch] graphics/danpei bug fixing, STAGE supporting.
Product: Ports & Packages Reporter: ru_M1cRO <rum1cro>
Component: Individual Port(s)Assignee: William Grzybowski <wg>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description ru_M1cRO 2013-11-08 10:00:00 UTC
[patch] graphics/danpei bug fixing, STAGE supporting.
Fixed building with clang
Added STAGE support

Fix: File was attached

Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-11-08 10:00:08 UTC
Class Changed
From-To: change-request->maintainer-update

Fix category (submitter is maintainer) (via the GNATS Auto Assign Tool)
Comment 2 William Grzybowski freebsd_committer freebsd_triage 2013-11-08 12:55:49 UTC
Responsible Changed
From-To: freebsd-ports-bugs->wg

I'll take it.
Comment 3 dfilter service freebsd_committer freebsd_triage 2013-11-08 13:35:21 UTC
Author: wg
Date: Fri Nov  8 13:35:07 2013
New Revision: 333221
URL: http://svnweb.freebsd.org/changeset/ports/333221

Log:
  graphics/danpei: allow staging
  
  - Allow staging
  - Add LICENSE (GPLv2)
  - Fix build with clang
  - Convert lib depends to new format [1]
  - Use IMAGEMAGICK as option [1]
  
  PR:		ports/183778
  Submitted by:	Ilya A. Arkhipov <rum1cro yandex.ru> (maintainer), wg [1]

Added:
  head/graphics/danpei/files/patch-viewer.c   (contents, props changed)
Deleted:
  head/graphics/danpei/pkg-plist
Modified:
  head/graphics/danpei/Makefile
  head/graphics/danpei/files/patch-image_cache.c

Modified: head/graphics/danpei/Makefile
==============================================================================
--- head/graphics/danpei/Makefile	Fri Nov  8 13:07:45 2013	(r333220)
+++ head/graphics/danpei/Makefile	Fri Nov  8 13:35:07 2013	(r333221)
@@ -10,11 +10,14 @@ MASTER_SITES=	SF
 MAINTAINER=	rum1cro@yandex.ru
 COMMENT=	Image Viewer for X Window System with thumbnail preview feature
 
-LIB_DEPENDS=	png15:${PORTSDIR}/graphics/png
+LICENSE=	GPLv2
 
-.if !defined(WITHOUT_IMAGEMAGICK)
-RUN_DEPENDS+=	convert:${PORTSDIR}/graphics/ImageMagick
-.endif
+LIB_DEPENDS=	libpng15.so:${PORTSDIR}/graphics/png
+
+OPTIONS_DEFINE=	IMAGEMAGICK
+OPTIONS_DEFAULT=IMAGEMAGICK
+
+IMAGEMAGICK_RUN_DEPENDS=	convert:${PORTSDIR}/graphics/ImageMagick
 
 USES=		gettext
 USE_GNOME=	gtk12 gdkpixbuf
@@ -23,5 +26,8 @@ CONFIGURE_ENV=	LIBS="-L${LOCALBASE}/lib"
 CPPFLAGS+=	-I${LOCALBASE}/include
 CONFIGURE_ARGS+=	--without-included-gettext
 
-NO_STAGE=	yes
+PLIST_FILES=	bin/danpei \
+		share/locale/de/LC_MESSAGES/danpei.mo \
+		share/locale/ja/LC_MESSAGES/danpei.mo
+
 .include <bsd.port.mk>

Modified: head/graphics/danpei/files/patch-image_cache.c
==============================================================================
--- head/graphics/danpei/files/patch-image_cache.c	Fri Nov  8 13:07:45 2013	(r333220)
+++ head/graphics/danpei/files/patch-image_cache.c	Fri Nov  8 13:35:07 2013	(r333221)
@@ -1,5 +1,27 @@
---- src/image_cache.c.orig	2003-08-08 09:17:58.000000000 +0200
-+++ src/image_cache.c	2012-05-03 23:39:36.000000000 +0200
+--- src/image_cache.c.orig	2013-11-07 19:24:24.056322038 +0400
++++ src/image_cache.c	2013-11-07 19:32:26.759288924 +0400
+@@ -356,7 +356,7 @@
+   save_file = NULL;
+   cl        = NULL;
+ 
+-  if (tp->cache.cache_dir == NULL) { return; }
++  if (tp->cache.cache_dir == NULL) { return FALSE; }
+ 
+   save_file = (gchar*)malloc(sizeof(gchar) * 
+                              (strlen(tp->cache.cache_dir) +
+@@ -761,9 +761,9 @@
+    /* Initialize the local variables. */
+    png_text_filename = buffer = NULL;
+ 
+-   if (pixbuf == NULL)   { return; }
+-   if (filename == NULL) { return; }
+-   if (strcmp(filename, "") == 0) { return; }
++   if (pixbuf == NULL)   { return FALSE; }
++   if (filename == NULL) { return FALSE; }
++   if (strcmp(filename, "") == 0) { return FALSE; }
+ 
+    fp = fopen (filename, "wb");
+    if (fp == NULL) { return FALSE; }
 @@ -782,7 +782,7 @@
        return FALSE;
     }
@@ -9,3 +31,12 @@
        png_destroy_write_struct(&png_ptr, &info_ptr);
        fclose (fp);
        return FALSE;
+@@ -839,7 +839,7 @@
+          /* expand RGB to RGBA using an opaque alpha value */
+          gint x;
+          gchar *buffer_ptr = buffer;
+-         gchar *source_ptr = pixels;
++         guchar *source_ptr = pixels;
+          for (x = 0; x < width; x++) {
+             *buffer_ptr++ = *source_ptr++;
+             *buffer_ptr++ = *source_ptr++;

Added: head/graphics/danpei/files/patch-viewer.c
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/graphics/danpei/files/patch-viewer.c	Fri Nov  8 13:35:07 2013	(r333221)
@@ -0,0 +1,29 @@
+--- src/viewer.c.orig	2013-11-07 19:33:02.807286361 +0400
++++ src/viewer.c	2013-11-07 19:34:22.533326686 +0400
+@@ -293,7 +293,7 @@
+ 
+   if ((dv->thumb == dv->current_thumb)  &&
+       ((widget->allocation.x == dv->area_x) &&
+-       (widget->allocation.y == dv->area_y)    )){ return; }
++       (widget->allocation.y == dv->area_y)    )){ return 1; }
+ 
+   dv->area_x = widget->allocation.x;
+   dv->area_y = widget->allocation.y;
+@@ -314,7 +314,7 @@
+       filename = g_strconcat(dv->thumb->path, "/", dv->thumb->filename, NULL);
+     }
+   }
+-  if (filename == NULL) { return; }
++  if (filename == NULL) { return 1; }
+ 
+   viewer_read_image(dv, filename, tp);
+ 
+@@ -555,7 +555,7 @@
+   while ((thumb != NULL) && (i < (tp->viewer).adj.value)) {
+     thumb = thumb->next; i++;
+   }
+-  if (thumb == NULL) { return; }
++  if (thumb == NULL) { return FALSE; }
+ 
+   gtk_window_set_title(GTK_WINDOW((tp->viewer).window), 
+     g_strconcat(thumb->path, "/", thumb->filename, NULL));
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
Comment 4 William Grzybowski freebsd_committer freebsd_triage 2013-11-08 13:37:23 UTC
State Changed
From-To: open->closed

Committed. Thanks!