[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] add fbida
From: |
Kei Kebreau |
Subject: |
Re: [PATCH] add fbida |
Date: |
Sat, 15 Oct 2016 16:51:00 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Julien Lepiller <address@hidden> writes:
> Hi,
>
> I attached a patch to add fbida, a frame-buffer based image and pdf
> viewer. I created a new file, since I didn't find an existing one
> where it could go. Let me know if you prefer it in an existing file.
Hi Julien! Your patch looks fine as far as linting and reproducibility
goes. Some pointers for your patch:
In my opinion, this can go in pdf.scm, as the mupdf package is there and
it has similar functions to fbida. Also, when you add fbida to pdf.scm,
don't forget to add a copyright notice for yourself!
signature.asc
Description: PGP signature