This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Review - Very minor issues] tcm

Harold L Hunt II wrote:
3) CYGWIN-PATCHES/tcm-2.20.README should be named CYWGIN-PATCHES/tcm.README. This isn't actually installed either, which requires that the "install" command in the build script be changed to the following:

install() {
  (cd ${objdir} && \
  make install prefix=${instdir} && \
  if [ ! -d ${instdir}${prefix}/X11R6/share/doc/Cygwin ]; then \
    mkdir -p ${instdir}${prefix}/X11R6/share/doc/Cygwin ; \
  fi && \
  if [ ! -d ${instdir}${prefix}/X11R6/share/doc/${PKG}-${VER} ]; then \
    mkdir -p ${instdir}${prefix}/X11R6/share/doc/${PKG}-${VER} ; \
  fi && \
  /usr/bin/install -m 644 ${srcdir}/CYGWIN-PATCHES/${PKG}.README \
    ${instdir}${prefix}/X11R6/share/doc/Cygwin/${PKG}-${VER}.README )
}

Oh yeah, and this bit should be removed from the Makefile (no need to have it there... should be stuck in the distribution build script (as I showed above) ).

Here is the line in the Makefile that should be removed:

$(TCM_INSTALL_CMD) CYGWIN-PATCHES/tcm-2.20.README /usr/share/doc/Cygwin/


Harold

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]