Committed in 11075. Thanks

Travis

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

>  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
> > [email protected]
> > http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
>
> Committed in 11064. Thanks
>
> Travis
> _______________________________________________
> openwrt-devel mailing [EMAIL 
> PROTECTED]://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
>
>
> _______________________________________________
> openwrt-devel mailing list
> [email protected]
> http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
>
>
_______________________________________________
openwrt-devel mailing list
[email protected]
http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to