public inbox for development@lists.ipfire.org
 help / color / mirror / Atom feed
From: Michael Tremer <michael.tremer@ipfire.org>
To: Adolf Belka <adolf.belka@ipfire.org>
Cc: "IPFire: Development-List" <development@lists.ipfire.org>
Subject: Re: [PATCH 1/6] lldpd: New package
Date: Fri, 7 Nov 2025 14:10:34 +0000	[thread overview]
Message-ID: <75B7320F-0C66-4EF9-8D39-AFC23714ABFD@ipfire.org> (raw)
In-Reply-To: <aa57a4f7-898a-4607-95e5-f9e7879f73b8@ipfire.org>

Hello Adolf,

> On 6 Nov 2025, at 21:45, Adolf Belka <adolf.belka@ipfire.org> wrote:
> 
> Reviewed-by: Adolf Belka <adolf.belka@ipfire.org>
> 
> Hi Michael,
> 
> llpd has a configure option --disable-static to remove the .a file from the rootfile.

Thank you for this one. I have applied this:

  https://git.ipfire.org/?p=ipfire-2.x.git;a=commitdiff;h=91a6323cfa0622a5e8cc198673515d858f36e220

> I am not sure if the doc and man files are doxygen based or not but if they are then there are also --disable-doxygen-doc and --disable-doxygen-man options, which would remove those from the rootfile also.

We don’t have doxygen, so they might just be static files which will just be installed.

> If you do edit the file then you could update the year for the LFS file from 2024 to 2025.

Well, I forgot about this one now. I never know what to do with those headers.

-Michael

> Regards,
> 
> Adolf.
> 
> On 06/11/2025 17:46, Michael Tremer wrote:
>> Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
>> ---
>>  config/rootfiles/common/lldpd | 25 ++++++++++
>>  lfs/lldpd                     | 87 +++++++++++++++++++++++++++++++++++
>>  make.sh                       |  1 +
>>  3 files changed, 113 insertions(+)
>>  create mode 100644 config/rootfiles/common/lldpd
>>  create mode 100644 lfs/lldpd
>> diff --git a/config/rootfiles/common/lldpd b/config/rootfiles/common/lldpd
>> new file mode 100644
>> index 000000000..7ead3d00b
>> --- /dev/null
>> +++ b/config/rootfiles/common/lldpd
>> @@ -0,0 +1,25 @@
>> +#etc/lldpd.d
>> +#etc/lldpd.d/README.conf
>> +#usr/include/lldp-const.h
>> +#usr/include/lldpctl.h
>> +#usr/include/lldpctl.hpp
>> +#usr/lib/liblldpctl.a
>> +#usr/lib/liblldpctl.la
>> +#usr/lib/liblldpctl.so
>> +usr/lib/liblldpctl.so.4
>> +usr/lib/liblldpctl.so.4.10.0
>> +#usr/lib/pkgconfig/lldpctl.pc
>> +usr/sbin/lldpcli
>> +usr/sbin/lldpctl
>> +usr/sbin/lldpd
>> +#usr/share/bash-completion/completions/lldpcli
>> +#usr/share/doc/lldpd
>> +#usr/share/doc/lldpd/CONTRIBUTE.md
>> +#usr/share/doc/lldpd/ChangeLog
>> +#usr/share/doc/lldpd/LICENSE
>> +#usr/share/doc/lldpd/NEWS
>> +#usr/share/doc/lldpd/README.md
>> +#usr/share/man/man8/lldpcli.8
>> +#usr/share/man/man8/lldpctl.8
>> +#usr/share/man/man8/lldpd.8
>> +#usr/share/zsh/site-functions/_lldpcli
>> diff --git a/lfs/lldpd b/lfs/lldpd
>> new file mode 100644
>> index 000000000..30302e823
>> --- /dev/null
>> +++ b/lfs/lldpd
>> @@ -0,0 +1,87 @@
>> +###############################################################################
>> +#                                                                             #
>> +# IPFire.org - A linux based firewall                                         #
>> +# Copyright (C) 2007-2024  IPFire Team  <info@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 <http://www.gnu.org/licenses/>.       #
>> +#                                                                             #
>> +###############################################################################
>> +
>> +###############################################################################
>> +# Definitions
>> +###############################################################################
>> +
>> +include Config
>> +
>> +VER        = 1.0.20
>> +
>> +THISAPP    = lldpd-$(VER)
>> +DL_FILE    = $(THISAPP).tar.gz
>> +DL_FROM    = $(URL_IPFIRE)
>> +DIR_APP    = $(DIR_SRC)/$(THISAPP)
>> +TARGET     = $(DIR_INFO)/$(THISAPP)
>> +
>> +###############################################################################
>> +# Top-level Rules
>> +###############################################################################
>> +
>> +objects = $(DL_FILE)
>> +
>> +$(DL_FILE) = $(DL_FROM)/$(DL_FILE)
>> +
>> +$(DL_FILE)_BLAKE2 = 95743f28d9b3c8ad6f354f7def5f835d9b0668c151ad429dccfc7a249e29234a9ca1fda6b3bcc2890c424053b5adf2d4d9d7c0cb2887e97cc32b42577b91c63a
>> +
>> +install : $(TARGET)
>> +
>> +check : $(patsubst %,$(DIR_CHK)/%,$(objects))
>> +
>> +download :$(patsubst %,$(DIR_DL)/%,$(objects))
>> +
>> +b2 : $(subst %,%_BLAKE2,$(objects))
>> +
>> +dist:
>> + $(PAK)
>> +
>> +###############################################################################
>> +# Downloading, checking, b2sum
>> +###############################################################################
>> +
>> +$(patsubst %,$(DIR_CHK)/%,$(objects)) :
>> + @$(CHECK)
>> +
>> +$(patsubst %,$(DIR_DL)/%,$(objects)) :
>> + @$(LOAD)
>> +
>> +$(subst %,%_BLAKE2,$(objects)) :
>> + @$(B2SUM)
>> +
>> +###############################################################################
>> +# Installation Details
>> +###############################################################################
>> +
>> +$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
>> + @$(PREBUILD)
>> + @rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar axf $(DIR_DL)/$(DL_FILE)
>> + cd $(DIR_APP) && ./configure \
>> + --prefix=/usr \
>> + --sysconfdir=/etc \
>> + --localstatedir=/var \
>> + --with-seccomp \
>> + --with-privsep-user=nobody \
>> + --with-privsep-group=nobody \
>> + --without-embedded-libevent
>> + cd $(DIR_APP) && make $(MAKETUNING)
>> + cd $(DIR_APP) && make install
>> + @rm -rf $(DIR_APP)
>> + @$(POSTBUILD)
>> diff --git a/make.sh b/make.sh
>> index b17e90e13..c542c8173 100755
>> --- a/make.sh
>> +++ b/make.sh
>> @@ -2125,6 +2125,7 @@ build_system() {
>>   lfsmake2 fort-validator
>>   lfsmake2 arpwatch
>>   lfsmake2 suricata-reporter
>> + lfsmake2 lldpd
>>     lfsmake2 linux
>>   lfsmake2 rtl8812au
> 



  reply	other threads:[~2025-11-07 14:10 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-06 16:46 New Feature: LLDP Michael Tremer
2025-11-06 16:46 ` [PATCH 1/6] lldpd: New package Michael Tremer
2025-11-06 21:45   ` Adolf Belka
2025-11-07 14:10     ` Michael Tremer [this message]
2025-11-06 16:46 ` [PATCH 2/6] misc-progs: Add lldpdctrl Michael Tremer
2025-11-06 16:46 ` [PATCH 3/6] cfgroot: Create /var/ipfire/lldp/settings Michael Tremer
2025-11-06 16:46 ` [PATCH 4/6] web-user-interface: Add lldp.cgi Michael Tremer
2025-11-06 16:46 ` [PATCH 5/6] web-user-interface: Add LLDP to the network menu Michael Tremer
2025-11-06 16:46 ` [PATCH 6/6] initscripts: Add an initscript for lldpd 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=75B7320F-0C66-4EF9-8D39-AFC23714ABFD@ipfire.org \
    --to=michael.tremer@ipfire.org \
    --cc=adolf.belka@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