Bug 101529 - editors/the can be built automatically for full X11 capabilities with a few minor changes
Summary: editors/the can be built automatically for full X11 capabilities with a few m...
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: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-08-06 22:20 UTC by User1001
Modified: 2006-08-31 20:48 UTC (History)
0 users

See Also:


Attachments
file.diff (38 bytes, patch)
2006-08-06 22:20 UTC, User1001
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description User1001 2006-08-06 22:20:13 UTC

By default, only the NCURSES version of THE is built (which is not very useful, really). To build the XCurses (X11) version requires the inclusion of devel/pdcurses XCurses.so(.8). Simple changes to the Makefile make this an automatic process with an additional "depends(WITH_XCURSES)" subsection. This builds and installs "xthe" instead of "nthe". "nthe" can still be built by default.

Note: I don't know how to properly adjust the pkg-plist for this conditional object build.

Fix: MAN1=         the.1

+ .if defined(WITH_XAW3D)
+ CONFIGURE_ARGS+=        --with-xaw3d
+ LIB_DEPENDS+=   Xaw3d.8:${PORTSDIR}/x11-toolkits/Xaw3d
+ .endif
+ .if defined(WITH_XCURSES)
+ CONFIGURE_ARGS+=      --with-xcurses
+ LIB_DEPENDS+= XCurses.8:${PORTSDIR}/devel/pdcurses
+ .endif
+
  .if defined(WITH_REXX_REGINA)
  BUILD_DEPENDS+=       regina:${PORTSDIR}/lang/rexx-regina
  RUN_DEPENDS+= regina:${PORTSDIR}/lang/rexx-regina
***************


post-install:
+ .if defined(WITH_XCURSES)
+       @${LN} -sf xthe ${PREFIX}/bin/the
+ .else
        @${LN} -sf nthe ${PREFIX}/bin/the
+ .endif
  .if !defined(NOPORTDOCS)
        @${MKDIR} ${PREFIX}/share/doc/THE
        ${INSTALL_DATA} ${WRKSRC}/*.html ${PREFIX}/share/doc/THE--Ti7kful0PPI679jQzk0vjMTyfvzIUnncyNmFRlJfQkwCwyCj
Content-Type: text/plain; name="file.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="file.diff"

*** Makefile.orig       Tue Jan 31 19:03:09 2006
--- Makefile    Sun Aug  6 16:00:09 2006
***************
*** 27,32 ****
--- 27,41 ----
Comment 1 User1001 2006-08-09 07:37:32 UTC
Please modify the previous changes for WITH_CURSES from

+ .if defined(WITH_XCURSES)
+ CONFIGURE_ARGS+=      --with-xcurses
+ LIB_DEPENDS+= XCurses.8:${PORTSDIR}/devel/pdcurses
+ .endif

to

+ .if defined(WITH_XCURSES)
+ CONFIGURE_ARGS+=      --with-xcurses \
+                       --with-cursesincdir=/usr/local/include \
+                       --with-curseslibdir=/usr/local/lib
+ LIB_DEPENDS+= XCurses.2:${PORTSDIR}/devel/pdcurses
+ .endif
Comment 2 Pav Lucistnik freebsd_committer freebsd_triage 2006-08-31 20:48:45 UTC
State Changed
From-To: open->closed

Committed, with improvements (plist conditional, OPTIONS menu)