[Bug 227368] New port: graphics/phpsane: web-based frontend for SANE written in HTML/PHP so you can scan with your web-browser

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=227368

Tobias Kortkamp <tobik at freebsd.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |needs-patch, needs-qa
             Status|New                         |Open

--- Comment #2 from Tobias Kortkamp <tobik at freebsd.org> ---
Hello,

is this still relevant?  I'm a little concerned that the last release
of phpSANE was in 2013.  Does it work correctly with PHP 7.x?

XMASTER_SITES=  SF/${PORTNAME}/${PORTSANE}/${PORTVERSION}

The distfile does not fetch.  What is ${PORTSANE} ?

XIMAGICK_RUN_DEPENDS=   convert:graphics/ImageMagick-nox11

graphics/ImageMagick-nox11 no longer exists.  You must pick either
of graphics/ImageMagick{6,7}-nox11.

XNO_ARCH=       yes
XNO_BUILD=      yes
XNO_WRKSUBDIR=  yes
X
XPLIST_SUB+=    PHPSANE_GROUP=${PHPSANE_GROUP} \
X               PHPSANE_TMPDIR=${PHPSANE_TMPDIR} \
X               PHPSANE_OUTPUTDIR=${PHPSANE_OUTPUTDIR} \
X               PHPSANE_SCANNERSDIR=${PHPSANE_SCANNERSDIR} \

These need to be before OPTIONS_DEFINE.  Also replace the += with
just =.

X       cd ${WRKSRC} ; \
X           ${FIND} * -type d -exec ${MKDIR} "${STAGEDIR}${WWWDIR}/{}" \;
X       cd ${WRKSRC} ; \
X           ${FIND} * ! -type d ! -name Thumbs.db ! -name '*.zip' \
X               ! -name '*.orig' -exec ${INSTALL_DATA} "{}" \
X               "${STAGEDIR}${WWWDIR}/{}" \;

Please use ${COPYTREE_SHARE} instead.

X       ${MKDIR} "${STAGEDIR}${PHPSANE_TMPDIR}"
X       ${MKDIR} "${STAGEDIR}${PHPSANE_OUTPUTDIR}"
X       ${MKDIR} "${STAGEDIR}${PHPSANE_SCANNERSDIR}"

No need for the "

X+$PNMTOJPEG = "/usr/local/bin/pnmtojpeg"; //netpbm pnm to jpeg conversion
binary
X+$PNMTOTIFF = "/usr/local/bin/pnmtotiff"; //netpbm pnm to tiff conversion
binary
X+$PNMTOBMP  = "/usr/local/bin/ppmtobmp";  //netpbm ppm to bmp conversion
binary
X+$PNMTOPNG  = "/usr/local/bin/pnmtopng";  //netpbm pnm to png conversion
binary
X+$CONVERT   = "/usr/local/bin/convert";   //ImageMagick convert binary
X+$IDENTIFY  = "/usr/local/bin/identify";  //ImageMagick binary used to test
for PDF support
X if(!`ls $GOCR`) $do_format_txt = false; //disable OCR when not available

/usr/local is user settable via LOCALBASE, so hardcoding it like
this is probably wrong.  Maybe add an additional

post-patch:
        @${REINPLACE_CMD} 's,/usr/local,${LOCALBASE},'
${WRKSRC}/incl/config.php

or do it completely with REINPLACE_CMD instead.

-- 
You are receiving this mail because:
You are the assignee for the bug.