I think having a -doc package could be useful so that people that want the
documentation can simply install it.

Travis

On Thu, May 8, 2008 at 8:52 AM, <[EMAIL PROTECTED]> wrote:

> hi
>
> why do we need -doc as a package ?
>
> blogic
>
>
>
>
>
>
> Quoting Mark Kelly <[EMAIL PROTECTED]>:
>
> > This patch is broken for me:
> >
> > Unpacking vim-full...Done.
> > Configuring
> >
> vim-full.../opt/openwrt/k1/build_dir/armeb/root-ixp4xx//usr/lib/ipkg/info/vim-full.postinst:
> 6: Syntax error: "fi" unexpected (expecting
> > "then")
> > make[3]: ***
> >
> [/opt/openwrt/k1/build_dir/armeb/root-ixp4xx/usr/lib/ipkg/info/vim-full.list]
> Error
> > 2
> > make[3]: Leaving directory `/opt/openwrt/packages/utils/vim'
> > make[2]: *** [package/vim/install] Error 2
> > make[2]: Leaving directory `/opt/openwrt/k1'
> > make[1]: *** [/opt/openwrt/k1/staging_dir/armeb/stamp/.package_install]
> > Error 2
> > make[1]: Leaving directory `/opt/openwrt/k1'
> > make: *** [world] Error 2
> >
> >
> >
> /opt/openwrt/k1/build_dir/armeb/root-ixp4xx//usr/lib/ipkg/info/vim-full.postinst
> should contain an extra "THEN" it appears, this patch fixes
> > it.
> > ?
> > Signed-off-by: Mark Kelly <mark . kelly at cadre5 . com>
> >
> > Index: Makefile
> > ===================================================================
> > ---? utils/vim/ Makefile      (revision 11074)
> > +++ utils/vim/Makefile        (working copy)
> > @@ -160,6 +160,7 @@
> >    if [ $${IPKG_INSTROOT} != / ]
> >    then
> >      if [ -d /usr/share/ ]
> > +    then
> >        ln -s $${IPKG_INSTROOT}/usr/share/vim/ /usr/share/
> >      fi
> >    fi
> > @@ -170,6 +171,7 @@
> >    if [ $${IPKG_INSTROOT} != / ]
> >    then
> >      if [ -d /usr/share/ ]
> > +    then
> >        ln -s $${IPKG_INSTROOT}/usr/share/vim/vimrc /usr/share/vim/vimrc
> >      fi
> >    fi
> >
> >
> >
> >
> >
> > On Wed, 2008-05-07 at 19:49 -0500, Travis Kemen wrote:
> >
> >> Michael wrote:
> >> > Here is a patch that adds a vim-full (normal options)  target to
> vim... as
> >> > well as split out vim-runtime and vim-doc.
> >> > The vim-runtime could probably be split further into base/ syntax
> >>   but given
> >> > that it requires 1.4 M vim-full I'm not too worried.
> >> >
> >> >
> >> > Signed-off-by: Michael Geddes <michael at frog.wheelycreek.net>
> >> >
> >> > Index: utils/vim/Makefile
> >> > ===================================================================
> >> > --- utils/vim/Makefile     (revision 10654)
> >> > +++ utils/vim/Makefile     (working copy)
> >> > @@ -20,7 +20,7 @@
> >> >
> >> >  include $(INCLUDE_DIR)/package.mk
> >> >
> >> > -define Package/vim
> >> > +define Package/vim/Default
> >> >    SECTION:=utils
> >> >    CATEGORY:=Utilities
> >> >    DEPENDS:=+libncurses
> >> > @@ -29,15 +29,59 @@
> >> >    SUBMENU:=editors
> >> >  endef
> >> >
> >> > +define Package/vim
> >> > +  $(call Packages/vim/Default)
> >> > +  TITLE+= (Tiny)
> >> > +endef
> >> > +
> >> > +define Package/vim-full
> >> > +  $(call Packages/vim/Default)
> >> > +  TITLE+= (Normal)
> >> > +endef
> >> > +
> >> > +define Package/vim-runtime
> >> > +  $(call Packages/vim/Default)
> >> > +  TITLE+= (runtime files)
> >> > +endef
> >> > +
> >> > +define Package/vim-help
> >> > +  $(call Packages/vim/Default)
> >> > +  TITLE+= (help files)
> >> > +endef
> >> > +
> >> > +define Package/vim-full/conffiles
> >> > +/usr/share/vim/vimrc
> >> > +endef
> >> > +
> >> > +define Package/vim/conffiles
> >> > +/usr/share/vim/vimrc
> >> > +endef
> >> > +
> >> >  define Package/vim/description
> >> >    Vim is an almost compatible version of the UNIX editor Vi.
> >> > +  (Tiny build)
> >> >  endef
> >> >
> >> > +define Package/vim-full/description
> >> > +  Vim is an almost compatible version of the UNIX editor Vi.
> >> > +  (Normal build)
> >> > +endef
> >> > +
> >> > +define Package/vim-runtime/description
> >> > +  Vim is an almost compatible version of the UNIX editor Vi.
> >> > +  (Runtime files)
> >> > +endef
> >> > +
> >> > +define Package/vim-help/description
> >> > +  Vim is an almost compatible version of the UNIX editor Vi.
> >> > +  (Help files)
> >> > +endef
> >> > +
> >> > +
> >> >  CONFIGURE_ARGS += \
> >> >    --disable-gui \
> >> >    --disable-gtktest \
> >> >    --disable-xim \
> >> > -  --with-features=tiny \
> >> >    --without-x \
> >> >    --disable-multibyte \
> >> >    --disable-netbeans \
> >> > @@ -45,11 +89,94 @@
> >> >    --disable-gpm \
> >> >    --with-tlib=ncurses
> >> >
> >> > -define Package/vim/install
> >> > +ifneq ($(CONFIG_PACKAGE_vim),)
> >> > +define Build/Compile/vim
> >> > +  $(call Build/Configure/Default, \
> >> > +    --with-features=tiny \
> >> > +   )
> >> > +  $(MAKE) -C $(PKG_BUILD_DIR) clean
> >> > +  $(MAKE) -C $(PKG_BUILD_DIR) \
> >> > +    DESTDIR="$(PKG_INSTALL_DIR)" \
> >> > +    all
> >> > +  $(CP) $(PKG_BUILD_DIR)/src/$(PKG_NAME) $(PKG_BUILD_DIR)/vim_tiny
> >> > +endef
> >> > +endif
> >> > +
> >> > +ifneq ($(CONFIG_PACKAGE_vim-full),)
> >> > +define Build/Compile/vim-full
> >> > +  $(call Build/Configure/Default, \
> >> > +    --with-features=normal \
> >> > +  )
> >> > +  $(MAKE) -C $(PKG_BUILD_DIR) clean
> >> > +  $(MAKE) -C $(PKG_BUILD_DIR) \
> >> > +    DESTDIR="$(PKG_INSTALL_DIR)" \
> >> > +    all
> >> > +  $(CP) $(PKG_BUILD_DIR)/src/$(PKG_NAME) $(PKG_BUILD_DIR)/vim_normal
> >> > +endef
> >> > +endif
> >> > +
> >> > +define Build/Compile/vim-runtime
> >> > +  $(MAKE) -C $(PKG_BUILD_DIR)/src DESTDIR="$(PKG_INSTALL_DIR)"
> >> installrtbase
> >> > +  tar -czf $(PKG_BUILD_DIR)/docs.tgz
> >> $(PKG_INSTALL_DIR)/usr/share/vim/vim*/doc
> >> > +  rm -rf $(PKG_INSTALL_DIR)/usr/share/vim/vim*/doc
> >> > +  rm -rf $(PKG_INSTALL_DIR)/usr/man
> >> > +endef
> >> > +
> >> > +define Build/Configure
> >> > +endef
> >> > +
> >> > +define Build/Compile
> >> > +$(call Build/Compile/vim)
> >> > +$(call Build/Compile/vim-full)
> >> > +$(call Build/Compile/vim-runtime)
> >> > +endef
> >> > +
> >> > +define Package/vim/install
> >> >    $(INSTALL_DIR) $(1)/usr/bin
> >> > -  $(INSTALL_BIN) $(PKG_BUILD_DIR)/src/$(PKG_NAME) $(1)/usr/bin/
> >> > +  $(INSTALL_BIN) $(PKG_BUILD_DIR)/vim_tiny $(1)/usr/bin/vim
> >> >    $(INSTALL_DIR) $(1)/usr/share/vim
> >> >    $(INSTALL_CONF) ./files/vimrc $(1)/usr/share/vim/
> >> >  endef
> >> >
> >> > +
> >> > +define Package/vim-full/install
> >> > +  $(INSTALL_DIR) $(1)/usr/bin
> >> > +  $(INSTALL_BIN) $(PKG_BUILD_DIR)/vim_normal $(1)/usr/bin/vim
> >> > +  $(INSTALL_DIR) $(1)/usr/share/vim
> >> > +  $(INSTALL_CONF) ./files/vimrc.full $(1)/usr/share/vim/vimrc
> >> > +endef
> >> > +
> >> > +define Package/vim-runtime/install
> >> > +  $(CP) $(PKG_INSTALL_DIR)/* $(1)
> >> > +  rm -rf $(1)/usr/share/vim/vim*/doc
> >> > +endef
> >> > +
> >> > +define Package/vim-help/install
> >> > +  tar -C $(1) -xzf $(PKG_BUILD_DIR)/docs.tgz
> >> > +endef
> >> > +
> >> > +define Package/vim-full/postinst
> >> > +#!/bin/sh
> >> > +  if [ $${IPKG_INSTROOT} != / ]
> >> > +  then
> >> > +    if [ -d /usr/share/ ]
> >> > +      ln -s $${IPKG_INSTROOT}/usr/share/vim/ /usr/share/
> >> > +    fi
> >> > +  fi
> >> > +endef
> >> > +
> >> > +define Package/vim/postinst
> >> > +#!/bin/sh
> >> > +  if [ $${IPKG_INSTROOT} != / ]
> >> > +  then
> >> > +    if [ -d /usr/share/ ]
> >> > +      ln -s $${IPKG_INSTROOT}/usr/share/vim/vimrc
> /usr/share/vim/vimrc
> >> > +    fi
> >> > +  fi
> >> > +endef
> >> > +
> >> >  $(eval $(call BuildPackage,vim))
> >> > +$(eval $(call BuildPackage,vim-full))
> >> > +$(eval $(call BuildPackage,vim-runtime))
> >> > +$(eval $(call BuildPackage,vim-help))
> >> > +
> >> > Index: utils/vim/files/vimrc.full
> >> > ===================================================================
> >> > --- utils/vim/files/vimrc.full
> >> > +++ utils/vim/files/vimrc.full     (working copy)
> >> > @@ -0,0 +1,23 @@
> >> > +set showcmd            " Show (partial) command in status line.
> >> > +set showmatch          " Show matching brackets.
> >> > +set ignorecase         " Do case insensitive matching
> >> > +set incsearch          " Incremental search
> >> > +set autowrite          " Automatically save before commands like
> :next
> >> > and :make
> >> > +set nocompatible        " Use Vim defaults instead of 100% vi
> >> compatibility
> >> > +set backspace=indent,eol,start  " more powerful backspacing
> >> > +set autoindent          " always set autoindenting on
> >> > +set linebreak         " Don't wrap words by default
> >> > +set textwidth=0         " Don't wrap lines by default
> >> > +set ruler               " show the cursor position all the time
> >> > +if has('syntax') && &term =~ "xterm"
> >> > +  if has("terminfo")
> >> > +    set t_Co=8
> >> > +    set t_Sf=<Esc>[3%p1%dm
> >> > +    set t_Sb=<Esc>[4%p1%dm
> >> > +  else
> >> > +    set t_Co=8
> >> > +    set t_Sf=<Esc>[3%dm
> >> > +    set t_Sb=<Esc>[4%dm
> >> > +  endif
> >> > +  syn on
> >> > +endif
> >> > _______________________________________________
> >> > openwrt-devel mailing list
> >> > openwrt-devel@lists.openwrt.org
> >> > http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
> >>
> >> Committed in 11064. Thanks
> >>
> >> Travis
> >> _______________________________________________
> >> openwrt-devel mailing list
> >> openwrt-devel@lists.openwrt.org
> >> http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
> >>
> >
>
>
>
> --
> John Crispin
> hacking, coding, etc
> blogic on irc
>
>    _______                     ________        __
>   |       |.-----.-----.-----.|  |  |  |.----.|  |_
>   |   -   ||  _  |  -__|     ||  |  |  ||   _||   _|
>   |_______||   __|_____|__|__||________||__|  |____|
>            |__| W I R E L E S S   F R E E D O M
>   KAMIKAZE (bleeding edge) -----------------------
>    * 10 oz Vodka       Shake well with ice and strain
>    * 10 oz Triple sec  mixture into 10 shot glasses.
>    * 10 oz lime juice  Salute!
>   ---------------------------------------------------
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
>
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to