Bug#682980: darktable: should use system shared libraw

2012-07-28 Thread Pascal de Bruijn
On Sat, Jul 28, 2012 at 6:19 PM, Jonas Smedegaard d...@jones.dk wrote:
 [switching to non-quiet flavor of bug address]

 On 12-07-28 at 12:09pm, Pascal de Bruijn wrote:
 On Sat, Jul 28, 2012 at 2:26 AM, Jonas Smedegaard d...@jones.dk wrote:
  On 12-07-28 at 12:42am, Pascal de Bruijn wrote:
  On Sat, Jul 28, 2012 at 12:28 AM, Jonas Smedegaard d...@jones.dk
  wrote:
   On 12-07-27 at 10:08pm, Pascal de Bruijn wrote:
   When Darktable is linked against an external Libraw (or for that
   matter RawSpeed) library, we likely would get lots of camera
   support bugs which aren't reproducible (assuming the Debian
   Libraw version is older), wasting our time. Or we aren't getting
   any valid camera support bugs reported (assuming the Debian
   Libraw version is newer). So both cases (newer and older) are
   detrimental to our project.
  
   Bugs from users of Debian should go to Debian for this exact
   reason: The Debian package maintainers should pass upstream to
   you the Darktable developers only bugs relevant for you.
 
  Yeah, but that's not reality. People will just come and ask on our
  mailing lists and irc channels, often not telling us they are
  running Debian (unless we specifically ask), wasting our time.
 
  I recognize that issue from users of Debian reporting bugs about
  packages derived from Debian but changed in various ways unknown to
  us.
 
  What I do with that is not try enforce one single use of the
  packages we provide, but a) tell our users that they are free to use
  Debian also in (to us) weird ways (that's one of the freedoms that
  DFSG-free licensing provides!), but b) they are strongly recommended
  to tell us very clearly up front when reporting bugs if their setup
  of Debian is unusual, to not waste our time e.g. chasing bugs
  inefficiently.

 I guess that's a similar issue.

 However, there is a difference with users personally modifying things.
 And distributions shipping non-standard versions.

 We'd like to make sure that users get a user experience that is
 representative of our intended Darktable user experience.

 Users of Debian are not only personal.  One user of Debian is the
 distribution Ubuntu.

Yes, which multiplies the problem for us. But at least for Ubuntu I
can point people to my Darktable PPAs.

   So in my opinion Darktable should get a permanent exception to
   this Debian policy.
  
   PS: Please don't misunderstand, I generally agree with the
   policy in this regard, it's just that it makes very little sense
   for projects like Darktable.
  
   Sorry, but I fail to see how this issue is any different from
   e.g. consumers of libexiv and the resulting changes to richness
   of the EXIF
 
  Having an older libexiv2 will not prevent files from being read at
  all. Having an old libraw could result in images being green
  instead of properly white balanced in some cases. And in even fewer
  cases it could result in files not loading at all (where they
  should have just loaded just fine (and/or not being green) with
  unpatched darktable sources).
 
   data supported with various versions of that library: as long as
   the API is the same, newest version of the library most often is
   preferred.
 
  Yes, but that isn't what happens in reality. What happens in
  reality is that Debian is usually behind, really...
 
   If I misunderstood and there is really something more intimate
   going on specifically with Darktable and its libraries could you
   please try elaborate more on that?
 
  With regard to the patch, LibRaw does RAW reading _and_ processing,
  we only use the RAW reading bits (which is fairly atypical). But
  the LibRaw processing bits don't support float DNGs (which we use
  for HDR IIRC), so the LibRaw authors are blocking them from being
  read. So we need to patch that up for our particular use.
 
  Besides the above, there is nothing more intimate going on, except
  that I see lots of potential problems, with little or no gain at
  all in our particular case.
 
  Thanks for the details.
 
  It still sound to me like Darktable would make good sense to link
  against shared libraries for Debian.

 I don't see how you'd resolve the float issue. But even if that were
 to be resolved. What is the perceived benefit in this particular case?

 Same benefits as with other cases.  This is nicely described in Debian
 Policy: http://www.debian.org/doc/debian-policy/footnotes.html#f30

1. Having multiple copies of the same code in Debian is inefficient

Ok. I get that. But it's hardly worth creating problems for.

2. often creates either static linking or shared library conflicts

I'm not sure how that would happen. Including local libraries
generally avoids that problem, instead of creating it.

3. and, most importantly, increases the difficulty of handling
security vulnerabilities in the duplicated code.

This is a good point, particularly for network/Internet facing program
and libraries.

Though for LibRaw it's a rather theoretical point

Bug#652060: Use lossless AdobeDeflate compression for TIFFs [BROKEN]

2011-12-16 Thread Pascal de Bruijn
Please do not apply c432c92723842e9e4c546fa1681c8604b3d791ca

While a similar patch has worked fine for UFRaw for years, when
applied to Darktable it seems to break 16 bit TIFF export.

Regards,
Pascal de Bruijn



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#608353: Gnoduino Debian/Ubuntu package

2011-07-02 Thread Pascal de Bruijn
Hi,

I'm not a very experienced python packager, but I have been working on
a package for Gnoduino. My packaging should be easy to import by the
Debian folks.

https://launchpad.net/~pmjdebruijn/+archive/gnoduino

Regards,
Pascal de Bruijn



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#553359: Upstream packaging

2010-07-24 Thread Pascal de Bruijn
I think most licensing issues have been resolved upstream.

We also maintain Ubuntu packages upstream:

  https://launchpad.net/~pmjdebruijn/+archive/ppa/+packages

So I don't see why Debian would want to package Darktable separately.

If there is anything wrong with my packages, please let me know, I'm
sure we'll be able to work things out, so Debian can import my package
1:1.

Regards,
Pascal de Bruijn



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#528748: libblkid1: device mapper devices aren't properly prioritized

2009-05-15 Thread Pascal de Bruijn | Unilogic Networks B.V.
Package: libblkid1
Version: 1.41.3-1

The current version of e2fsprogs/libblkid1 will show dm-multipath
devices as /dev/dm-X devices, instead of /dev/mapper/mpathX. This is a
cosmetic bug. However because the prioritization code in libblkid1 works
on device names, this leads to a lack of device prioritization.

The device prioritization is important when working with dm-multipath
and volume labels. If a dm/mpath device doesn't get prioritized, 'mount
-L' tends to mount members devices like /dev/sda instead of /dev/dm-X
or /dev/mapper/mpathX.

I've attached a patch, which correctly makes blkid return
the /dev/mapper/mpathX device instead of /dev/dm-X.

This change automatically fixes the prioritization code, because now the
devices names match.

This patch, is basically a rebased version of the following:
  
http://git.kernel.org/?p=fs/ext2/e2fsprogs.git;a=commit;h=4271e23942bdc60e1fa6c0b26bc666a94a8b3e1d
and
  http://thread.gmane.org/gmane.comp.file-systems.ext4/12896

-- 
Regards,

Pascal de Bruijn pas...@unilogicnetworks.net
Junior Network Engineer - Unilogic Networks B.V.

Unilogic Networks B.V.
Bergerweg 120
6135 KD Sittard
T: 046-4571640 
F: 046-4571641
E: i...@unilogicnetworks.net
diff -Nurpd e2fsprogs-1.41.3-orig/lib/blkid/devname.c e2fsprogs-1.41.3-mpath/lib/blkid/devname.c
--- e2fsprogs-1.41.3-orig/lib/blkid/devname.c	2008-10-07 16:22:39.0 +0200
+++ e2fsprogs-1.41.3-mpath/lib/blkid/devname.c	2009-05-15 08:55:28.0 +0200
@@ -177,6 +177,15 @@ static void probe_one(blkid_cache cache,
 	if (dev  dev-bid_devno == devno)
 		goto set_pri;
 
+	/* Try to translate private device-mapper dm-N names
+	 * to standard /dev/mapper/name.
+	 */
+	if (!strncmp(ptname, dm-, 3)  isdigit(ptname[3])) {
+		blkid__scan_dir(/dev/mapper, devno, 0, devname);
+		if (devname)
+			goto get_dev;
+	}
+
 	/*
 	 * Take a quick look at /dev/ptname for the device number.  We check
 	 * all of the likely device directories.  If we don't find it, or if
@@ -195,7 +204,7 @@ static void probe_one(blkid_cache cache,
 		if (stat(device, st) == 0  S_ISBLK(st.st_mode) 
 		st.st_rdev == devno) {
 			devname = blkid_strdup(device);
-			break;
+			goto get_dev;
 		}
 	}
 	/* Do a short-cut scan of /dev/mapper first */
@@ -206,6 +215,8 @@ static void probe_one(blkid_cache cache,
 		if (!devname)
 			return;
 	}
+
+get_dev:
 	dev = blkid_get_dev(cache, devname, BLKID_DEV_NORMAL);
 	free(devname);