Re: NEW: xournal-0.4.5

2011-10-10 Thread Stuart Henderson
On 2011-10-10, Anthony J. Bentley  wrote:
> Minor changes to Ido's xournal port from last May
> (http://marc.info/?l=openbsd-ports&m=130446181021112&w=2):
>
> - cleaned up WANTLIB (quiets port-lib-depends-check)
> - removed MAINTAINER (by request)

that's still there..

> - tweaked pkg/DESCR
>
> ok?

ports-wise OK and it looks like a nice thing to have, but on most files
I've tried (this is on amd64, haven't tried other arch), if I add some
text and "export as pdf" it dies:-

GLib-ERROR **: gmem.c:239: failed to allocate 17179869184 bytes
Trace/BPT trap (core dumped) 

(if I just add drawings without text, it exports ok)

example file which it fails on (but almost everything I've tried the
same happens) - http://www.openbsd.org/papers/slack2k11-on_compat_linux.pdf
steps to repeat: click the T icon - click somewhere - type something -
export as pdf - choose filename - boom.




Re: NEW: xournal-0.4.5

2011-10-09 Thread Anthony J. Bentley
Minor changes to Ido's xournal port from last May
(http://marc.info/?l=openbsd-ports&m=130446181021112&w=2):

- cleaned up WANTLIB (quiets port-lib-depends-check)
- removed MAINTAINER (by request)
- tweaked pkg/DESCR

ok?

--
Anthony J. Bentley


xournal.tar.gz
Description: GNU Zip compressed data


Re: NEW: xournal-0.4.5

2011-05-03 Thread Ido Admon


It was a peachy Sunday, May  1 2011, 17:08:28 when Ido Admon
 wrote:

> It was a peachy Sunday, May  1 2011, 12:48:25 when Stuart Henderson
>  wrote:
> > On 2011-05-01, Ido Admon  wrote:
> > > 1. The desktop icons etc. that some people may want are installed
> > > in a separate makefile target, that must be called explicitly.
> > > There must be a more elegant way of doing that than the one I've
> > > found (patching the Makefile).
> > 
> > INSTALL_TARGET (the analogue for building is ALL_TARGET).
> > 
> > > 2. Two source files can't seem to find the poppler header that
> > > they're trying to #include (they have no trouble finding gtk
> > > etc.). Again, my crude solution of patching them must have a
> > > better alternative.
> > 
> > Try something like CONFIGURE_ARGS= CPPFLAGS=-I${LOCALBASE}/include
> > 
> > > 3. I put this under textproc - would x11 be a better fit?
> > 
> > Either is probably ok, x11 might be slightly better
> > 
> > Other notes
> > 
> > - DESCR should be wrapped, maybe trim things a bit - I'd go for
> > something like this:
> > 
> > Xournal is an application for notetaking, sketching and keeping a
> > journal using a stylus. It is similar to Microsoft Windows Journal
> > or to other alternatives such as Jarnal, Gournal, and NoteLab.
> > 
> > It can also be used to annotate PDF files (highlight, underline
> > etc.), and either save the annotations in a separate file, or
> > export the annotated document as a new PDF file.
> > 
> > - patches should be generated with "make update-patches" (though the
> > patches you have now are unnecessary)
> > 
> > - needs some @exec in the PLIST for icons and desktop files and
> > a run dependency on desktop-file-utils
> > 
> > - use CONFIGURE_STYLE=gnu, remove BUILD_DEPENDS and
> > AUTOCONF_VERSION, these are used when you have to regenerate the
> > configure script whichs you aren't doing
> > 
> 
> Here's the prettier version. One patch remains necessary, since a path
> to /usr/share (instead of /usr/local/share) is hardcoded in the
> makefile.
> 
> Thanks Stuart for the help!

Format DESCR properly this time.

xournal-0.4.5.tgz
Description: application/compressed-tar


Re: NEW: xournal-0.4.5

2011-05-01 Thread Ido Admon
It was a peachy Sunday, May  1 2011, 12:48:25 when Stuart Henderson
 wrote:
> On 2011-05-01, Ido Admon  wrote:
> > 1. The desktop icons etc. that some people may want are installed in
> > a separate makefile target, that must be called explicitly. There
> > must be a more elegant way of doing that than the one I've found
> > (patching the Makefile).
> 
> INSTALL_TARGET (the analogue for building is ALL_TARGET).
> 
> > 2. Two source files can't seem to find the poppler header that
> > they're trying to #include (they have no trouble finding gtk etc.).
> > Again, my crude solution of patching them must have a better
> > alternative.
> 
> Try something like CONFIGURE_ARGS= CPPFLAGS=-I${LOCALBASE}/include
> 
> > 3. I put this under textproc - would x11 be a better fit?
> 
> Either is probably ok, x11 might be slightly better
> 
> Other notes
> 
> - DESCR should be wrapped, maybe trim things a bit - I'd go for
> something like this:
> 
> Xournal is an application for notetaking, sketching and keeping a
> journal using a stylus. It is similar to Microsoft Windows Journal or
> to other alternatives such as Jarnal, Gournal, and NoteLab.
> 
> It can also be used to annotate PDF files (highlight, underline etc.),
> and either save the annotations in a separate file, or export the
> annotated document as a new PDF file.
> 
> - patches should be generated with "make update-patches" (though the
> patches you have now are unnecessary)
> 
> - needs some @exec in the PLIST for icons and desktop files and
> a run dependency on desktop-file-utils
> 
> - use CONFIGURE_STYLE=gnu, remove BUILD_DEPENDS and AUTOCONF_VERSION,
> these are used when you have to regenerate the configure script whichs
> you aren't doing
> 

Here's the prettier version. One patch remains necessary, since a path
to /usr/share (instead of /usr/local/share) is hardcoded in the
makefile.

Thanks Stuart for the help!

xournal-0.4.5.tgz
Description: application/compressed-tar


Re: NEW: xournal-0.4.5

2011-05-01 Thread Stuart Henderson
On 2011-05-01, Ido Admon  wrote:
> 1. The desktop icons etc. that some people may want are installed in
> a separate makefile target, that must be called explicitly. There must
> be a more elegant way of doing that than the one I've found (patching
> the Makefile).

INSTALL_TARGET (the analogue for building is ALL_TARGET).

> 2. Two source files can't seem to find the poppler header that they're
> trying to #include (they have no trouble finding gtk etc.). Again, my
> crude solution of patching them must have a better alternative.

Try something like CONFIGURE_ARGS= CPPFLAGS=-I${LOCALBASE}/include

> 3. I put this under textproc - would x11 be a better fit?

Either is probably ok, x11 might be slightly better

Other notes

- DESCR should be wrapped, maybe trim things a bit - I'd go for
something like this:

Xournal is an application for notetaking, sketching and keeping a journal
using a stylus. It is similar to Microsoft Windows Journal or to other
alternatives such as Jarnal, Gournal, and NoteLab.

It can also be used to annotate PDF files (highlight, underline etc.),
and either save the annotations in a separate file, or export the
annotated document as a new PDF file.

- patches should be generated with "make update-patches" (though the
patches you have now are unnecessary)

- needs some @exec in the PLIST for icons and desktop files and
a run dependency on desktop-file-utils

- use CONFIGURE_STYLE=gnu, remove BUILD_DEPENDS and AUTOCONF_VERSION,
these are used when you have to regenerate the configure script whichs
you aren't doing




NEW: xournal-0.4.5

2011-05-01 Thread Ido Admon
"Xournal is an application for notetaking, sketching, keeping a journal
using a stylus."

More usefully, for me at least, is the ability to annotate PDF files
(underline, highlight etc.), something we currently only have in
okular, part of KDE4.

A few question regarding the port:
1. The desktop icons etc. that some people may want are installed in
a separate makefile target, that must be called explicitly. There must
be a more elegant way of doing that than the one I've found (patching
the Makefile).

2. Two source files can't seem to find the poppler header that they're
trying to #include (they have no trouble finding gtk etc.). Again, my
crude solution of patching them must have a better alternative.

3. I put this under textproc - would x11 be a better fit?


Thanks,
Ido

xournal-0.4.5.tgz
Description: application/compressed-tar