public inbox for development@lists.ipfire.org
 help / color / mirror / Atom feed
From: Jonatan Schlag <jonatan.schlag@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH 3/3] New package ncat (ncat is the new standard netcat) and change the build order of netcat
Date: Wed, 06 Apr 2016 10:26:23 +0000	[thread overview]
Message-ID: <20160406102623.Horde.HP9fj6fZA4iXVDYdvLBR7Xf@owncloud.helmsklamm.local.familyschlag> (raw)
In-Reply-To: <1459894194.30749.350.camel@ipfire.org>

[-- Attachment #1: Type: text/plain, Size: 6771 bytes --]

Hi,
it is necessary because the log files are named like  
$(THISAPP)-$(VER).  When in Ncat and Nmap
$(THISAPP)-$(VER) = nmap-7.12 is or the strings in both lfs files  
(ncat and nmap ) are the same, the log files are named identically. So  
the log file of the nmap package overrides the log file of the ncat  
package  .This should not happen.
The easiest solution till now was to change the name of the source  
tarball to ncat.

Or a way which I discover just now. Maybe it is a solution to change this to

VER        = 7.12

THISAPP    = ncat-$(VER)
DL_FILE    = nmap-$(VER).tar.bz2
DL_FROM  = $(URL_IPFIRE)
DIR_APP    = $(DIR_SRC)/nmap-$(VER)
TARGET     = $(DIR_INFO)/nmap-$(VER)
I will try this solution and if it works will send a new patch set.  
This solution is much more elegant :-).

Regards Jonatan

Michael Tremer – Mi, 6. April 2016 0:09
> Hi,
>
> why is it necessary to change the name of the tarball? It can be the  
> same as in
> lfs/nmap without any conflicts (as long as the checksum matches of  
> course which
> it should).
>
> -Michael
>
> On Mon, 2016-04-04 at 17:19 +0200, Jonatan Schlag wrote:
> > Signed-off-by: Jonatan Schlag <jonatan.schlag(a)ipfire.org>
> > ---
> >  config/rootfiles/packages/ncat |  5 +++
> >  lfs/ncat                       | 86
> > ++++++++++++++++++++++++++++++++++++++++++
> >  make.sh                        |  3 +-
> >  3 files changed, 93 insertions(+), 1 deletion(-)
> >  create mode 100644 config/rootfiles/packages/ncat
> >  create mode 100644 lfs/ncat
> > > diff --git a/config/rootfiles/packages/ncat  
> b/config/rootfiles/packages/ncat
> > new file mode 100644
> > index 0000000..f1cf714
> > --- /dev/null
> > +++ b/config/rootfiles/packages/ncat
> > @@ -0,0 +1,5 @@
> > +usr/bin/nc
> > +usr/bin/ncat
> > +#usr/share/man/man1/ncat.1
> > +#usr/share/ncat
> > +#usr/share/ncat/ca-bundle.crt
> > diff --git a/lfs/ncat b/lfs/ncat
> > new file mode 100644
> > index 0000000..bf02605
> > --- /dev/null
> > +++ b/lfs/ncat
> > @@ -0,0 +1,86 @@
> >  
> +#############################################################################
> > ##
> > +#                                                                  
>            >  #
> > +# IPFire.org - A linux based
> > firewall                                         #
> > +# Copyright (C) 2016 IPFire Team  <info(a)ipfire.org>                
>            >  #
> > +#                                                                  
>            >  #
> > +# This program is free software: you can redistribute it and/or
> > modify        #
> > +# it under the terms of the GNU General Public License as published
> > by        #
> > +# the Free Software Foundation, either version 3 of the License,
> > or           #
> > +# (at your option) any later
> > version.                                         #
> > +#                                                                  
>            >  #
> > +# This program is distributed in the hope that it will be
> > useful,             #
> > +# but WITHOUT ANY WARRANTY; without even the implied warranty
> > of              #
> > +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See
> > the               #
> > +# GNU General Public License for more
> > details.                                #
> > +#                                                                  
>            >  #
> > +# You should have received a copy of the GNU General Public
> > License           #
> > +# along with this program.  If not, see <www.gnu.org/licenses/>.      >  #
> > +#                                                                  
>            >  #
> >  
> +#############################################################################
> > ##
> > +
> >  
> +#############################################################################
> > ##
> > +# Definitions
> >  
> +#############################################################################
> > ##
> > +
> > +include Config
> > +
> > +VER        = 7.12
> > +
> > +THISAPP    = ncat-$(VER)
> > +DL_FILE    = $(THISAPP).tar.bz2
> > +DL_FROM    = $(URL_IPFIRE)
> > +DIR_APP    = $(DIR_SRC)/$(THISAPP)
> > +TARGET     = $(DIR_INFO)/$(THISAPP)
> > +PROG       = ncat
> > +PAK_VER    = 1
> > +
> > +DEPS       = ""
> > +
> >  
> +#############################################################################
> > ##
> > +# Top-level Rules
> >  
> +#############################################################################
> > ##
> > +
> > +objects = $(DL_FILE)
> > +
> > +$(DL_FILE) = $(DL_FROM)/$(DL_FILE)
> > +
> > +$(DL_FILE)_MD5 = 893d3722ecbd92ab523cb092c07e31d7
> > +
> > +install : $(TARGET)
> > +
> > +check : $(patsubst %,$(DIR_CHK)/%,$(objects))
> > +
> > +download :$(patsubst %,$(DIR_DL)/%,$(objects))
> > +
> > +md5 : $(subst %,%_MD5,$(objects))
> > +
> > +dist:
> > +	@$(PAK)
> > +
> >  
> +#############################################################################
> > ##
> > +# Downloading, checking, md5sum
> >  
> +#############################################################################
> > ##
> > +
> > +$(patsubst %,$(DIR_CHK)/%,$(objects)) :
> > +	@$(CHECK)
> > +
> > +$(patsubst %,$(DIR_DL)/%,$(objects)) :
> > +	@$(LOAD)
> > +
> > +$(subst %,%_MD5,$(objects)) :
> > +	@$(MD5)
> > +
> >  
> +#############################################################################
> > ##
> > +# Installation Details
> >  
> +#############################################################################
> > ##
> > +
> > +$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
> > +	@$(PREBUILD)
> > +	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar jxf $(DIR_DL)/$(DL_FILE)
> > +	cd $(DIR_APP) && mkdir -p build
> > +	cd $(DIR_APP) && ./configure --prefix=/usr --without-nmapfe --
> > without-zenmap --without-ndiff --without-nping -without-liblua  
> --without-nmap-
> > update
> > +	cd $(DIR_APP) && make $(MAKETUNING) $(EXTRA_MAKE)
> > +	cd $(DIR_APP)/ncat && make install
> > +	ln -f -s /usr/bin/ncat /usr/bin/nc
> > +	@rm -rf $(DIR_APP)
> > +	@$(POSTBUILD)
> > diff --git a/make.sh b/make.sh
> > index b32dfd2..ff1e404 100755
> > --- a/make.sh
> > +++ b/make.sh
> > @@ -679,6 +679,8 @@ buildipfire() {
> >    ipfiremake libevent2
> >    ipfiremake portmap
> >    ipfiremake nfs
> > +  ipfiremake netcat
> > +  ipfiremake ncat
> >    ipfiremake nmap
> >    ipfiremake ncftp
> >    ipfiremake etherwake
> > @@ -749,7 +751,6 @@ buildipfire() {
> >    ipfiremake gutenprint
> >    ipfiremake apcupsd
> >    ipfiremake iperf
> > -  ipfiremake netcat
> >    ipfiremake 7zip
> >    ipfiremake lynis
> >    ipfiremake streamripper
>


  reply	other threads:[~2016-04-06 10:26 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-04 15:19 Libvirt and Ncat/Netcat Jonatan Schlag
2016-04-04 15:19 ` [PATCH 1/3] Remove the nc symlink from netcat package (gnu-netcat is no longer our standard netcat) Jonatan Schlag
2016-04-04 15:19 ` [PATCH 2/3] Build nmap without ncat Jonatan Schlag
2016-04-04 15:19 ` [PATCH 3/3] New package ncat (ncat is the new standard netcat) and change the build order of netcat Jonatan Schlag
2016-04-05 22:09   ` Michael Tremer
2016-04-06 10:26     ` Jonatan Schlag [this message]
2016-04-06 16:50 Jonatan Schlag
2016-04-06 21:03 Michael Tremer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160406102623.Horde.HP9fj6fZA4iXVDYdvLBR7Xf@owncloud.helmsklamm.local.familyschlag \
    --to=jonatan.schlag@ipfire.org \
    --cc=development@lists.ipfire.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox