Bug 168131 - New port: textproc/freexl FreeXL is an open source library to extract valid data from within an
Summary: New port: textproc/freexl FreeXL is an open source library to extract valid d...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Michael Scheidell
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-19 10:50 UTC by Loïc Bartoletti
Modified: 2012-05-24 02:33 UTC (History)
0 users

See Also:


Attachments
file.shar (2.11 KB, text/plain)
2012-05-19 10:50 UTC, Loïc Bartoletti
no flags Details
freexl.shar (2.39 KB, text/plain; charset=UTF-8)
2012-05-20 10:13 UTC, Loïc Bartoletti
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Loïc Bartoletti freebsd_committer freebsd_triage 2012-05-19 10:50:01 UTC
FreeXL is an open source library to extract valid data from within an Excel (.xls) spreadsheet.
FreeXL design goals:

    to be simple and lightweight
    to be stable, robust and efficient
    to be easily and universally portable
    completely ignoring any GUI-related oddity

Note that the final goal means that FreeXL ignores at all fonts, sizes and alignments, and most formats. It ignores Pivot Table, Charts, Formulas, Visual Basic macros and so on. FreeXL is structurally simple and quite light-weight (typically 40-80K of object code, stripped).

FreeXL is licensed under the MPL tri-license terms; you are free to choose the best-fit license between:

    the MPL 1.1
    the GPL v2.0 or any subsequent version
    the LGPL v2.1 or any subsequent version


FreeXL is needed for spatialite-3.0.1 (Spatial extension for SQlite3)

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2012-05-19 10:50:12 UTC
Class Changed
From-To: sw-bug->change-request

Fix category (new ports should be change-requests) (via the GNATS Auto 
Assign Tool)
Comment 2 Loïc Bartoletti freebsd_committer freebsd_triage 2012-05-20 10:13:41 UTC
I made a mistake with the patch (i forgot the plist...).

Here is the good.


Thanks
Comment 3 Michael Scheidell freebsd_committer freebsd_triage 2012-05-20 12:12:59 UTC
Responsible Changed
From-To: freebsd-ports-bugs->scheidell

I'll take it.
Comment 4 Michael Scheidell freebsd_committer freebsd_triage 2012-05-20 12:19:06 UTC
State Changed
From-To: open->feedback

Thank you for your new port submission. 
Developers like you are what help keep FreeBSD the OS of choice for serious server operators. 

A couple of things, mostly housekeeping: 
#1, the 'header' of the Makefile should be like this on any new port: 
# New ports collection makefile for:    freexl     
# Date created:                         19 May 2012  
# Whom:                                 lbartoletti <coder@tuxfamily.org> 
# 
# $FreeBSD$ 
# 

(change namw and date created) 
#2: 
run portlint -Nabt on your port, fix lint errors: (like this one) 
FATAL: /usr/home/scheidell/ports/ports/textproc/freexl/pkg-plist: [6]: installing pkg-config files into lib/pkgconfig.  All  
pkg-config files must be installed into libdata/pkgconfig for them to be found by pkg-config. 

#3: even if this works with a program that is in database category, it should be in the textproc catagory 
(ie: /usr/ports/textproc/freexl) 

I have made the above three changes and will be running this in a tinderbox and will let you know results. 
As pr ports/168153 depends on this port, I have cc'd the maintainer
Comment 5 Michael Scheidell freebsd_committer freebsd_triage 2012-05-20 12:47:43 UTC
  got to fix this:  linux(ish) likes ../lib/pkgconfig.  FreeBSD needs 
../libdata/pkgconfig.  You need to edit makefile,
(don't forget to fix pkg-plist, and use dirrmtry, not dirrm on 
libdata/pkgconfig:

see lines 350 and below for hints.

http://lorie.secnap.net/tb/index.php?action=display_markup_log&build=7-scheidell_I386&id=1873#350

hint: use REPINPLACE (sed) to change lib/pkgconfig to libdata/pkgconfig


-- 
Michael Scheidell, CTO
 >*| * SECNAP Network Security Corporation
d: +1.561.948.2259
w: http://people.freebsd.org/~scheidell
Comment 6 Michael Scheidell freebsd_committer freebsd_triage 2012-05-20 20:17:59 UTC
Last two things:
#1, we (as in the 'they') don't like PORTVERSION' s like '1.0.0b'.  
seems '1.0.0.b' is more standard for freebsd, so,
DISTVERSION=    1.0.0b
#PORTVERSION= (sorta distversion freebsdized, see 'make -V PORTVERSION' 
for real portversion, 1.0.0b becomes 1.0.0.b)

#2
post-patch:
         @${REINPLACE_CMD} -e 
's|(libdir)/pkgconfig|(prefix)/libdata/pkgconfig|g' \
                 ${WRKSRC}/Makefile.in


I will be running the last two in tindebox, then I will commit changes.  
you need to make sure you test it with pr ports/168153 once I commit it.

I will leave this pr open for now, and will be ccing owner of spatialite


-- 
Michael Scheidell, CTO
 >*| * SECNAP Network Security Corporation
d: +1.561.948.2259
w: http://people.freebsd.org/~scheidell
Comment 7 dfilter service freebsd_committer freebsd_triage 2012-05-20 20:31:24 UTC
scheidell    2012-05-20 19:31:10 UTC

  FreeBSD ports repository

  Modified files:
    textproc             Makefile 
  Added files:
    textproc/freexl      Makefile distinfo pkg-descr pkg-plist 
  Log:
  Add port: textproc/freexl
  FreeXL is an open source library to extract valid data from within an Excel
  (.xls) spreadsheet.
  FreeXL design goals:
  
      to be simple and lightweight
      to be stable, robust and efficient
      to be easily and universally portable
      completely ignoring any GUI-related oddity
  
  Note that the final goal means that FreeXL ignores at all fonts, sizes and
  alignments, and most formats. It ignores Pivot Table, Charts, Formulas, Visual
  Basic macros and so on. FreeXL is structurally simple and quite light-weight
  (typically 40-80K of object code, stripped).
  
  - Fix PORTVERSION to be more standard (SET DISTVERSION, have PORTVERSION auto set) [2]
  - Fix Makefile and pkg-plist, pkhconfig must be in libdata/* not lib/* [2]
  - Fix category [2]
  
  PR:             ports/168131 [1]
  Submitted by:   lbartoletti <coder@tuxfamily.org> [1]
  Reviewed by:    scheidell@ (me) [2]
  
  Revision  Changes    Path
  1.1722    +1 -0      ports/textproc/Makefile
  1.1       +29 -0     ports/textproc/freexl/Makefile (new)
  1.1       +2 -0      ports/textproc/freexl/distinfo (new)
  1.1       +13 -0     ports/textproc/freexl/pkg-descr (new)
  1.1       +6 -0      ports/textproc/freexl/pkg-plist (new)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 8 Michael Scheidell freebsd_committer freebsd_triage 2012-05-20 20:33:52 UTC
State Changed
From-To: feedback->closed
Comment 9 Michael Scheidell freebsd_committer freebsd_triage 2012-05-20 20:34:13 UTC
State Changed
From-To: closed->suspended

Suspended until wen@ completes pr ports/168153 
Wen has my permission to commit any changes he needs to, and is welcome to close this pr when he is happy with it.
Comment 10 Michael Scheidell freebsd_committer freebsd_triage 2012-05-20 23:13:01 UTC
State Changed
From-To: suspended->patched

Change synopsis, set to patched, waiting for wen@'s approval of the dependent port.
Comment 11 wenheping 2012-05-22 03:32:50 UTC
Sorry for the delay.
databases/spatialite depends on textproc/freexl, so please test and commit
this PR, then I could test and commit PR/168153.
Thanks.

wen
Comment 12 Wen Heping freebsd_committer freebsd_triage 2012-05-24 02:33:54 UTC
State Changed
From-To: patched->closed

New port added. It work well with spatialite. Thank you , all!