Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at
Bug 911781 (bittwist) - Review Request: bittwist - Bit-Twist is a simple yet powerful libpcap-based Ethernet packet generator.
Summary: Review Request: bittwist - Bit-Twist is a simple yet powerful libpcap-based E...
Alias: bittwist
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2013-02-15 23:37 UTC by Giuseppe Marco Randazzo
Modified: 2020-04-29 11:53 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2020-04-28 15:57:07 UTC
Type: ---

Attachments (Terms of Use)
bittwist new spec file (deleted)
2014-06-23 19:02 UTC, Giuseppe Marco Randazzo
no flags Details
bittwist-Makefile patch (deleted)
2014-06-23 19:04 UTC, Giuseppe Marco Randazzo
no flags Details | Diff
bittwist-2.0-2 SRC RPM 2014-07-15 (deleted)
2014-07-15 12:42 UTC, Giuseppe Marco Randazzo
no flags Details
bittwist spec file 2014-07-15 (deleted)
2014-07-15 12:43 UTC, Giuseppe Marco Randazzo
no flags Details
bittwist Make file patch 2014-07-15 (deleted)
2014-07-15 12:44 UTC, Giuseppe Marco Randazzo
no flags Details | Diff

Description Giuseppe Marco Randazzo 2013-02-15 23:37:43 UTC
Spec URL:

Description: Bit-Twist is a simple yet powerful libpcap-based Ethernet packet generator. It is designed to complement tcpdump, which by itself has done a great job at capturing network traffic.
With Bit-Twist, you can now regenerate your captured traffic onto a live network! Packets are generated from tcpdump trace file (.pcap file). Bit-Twist also comes with a comprehensive trace file editor to allow you to change the contents of a trace file.

Generally, packet generator is useful in simulating networking traffic or scenario, testing firewall, IDS, and IPS, and troubleshooting various network problems.

Fedora Account System Username: I do not have any account... i'm not able to make account because in the register form there is an error...

Comment 1 Eduardo Echeverria 2013-02-16 06:37:28 UTC
Hi Giuseppe 

About review process

Perhaps you have tried to register at any time when the platform was not available, however, and the expectation that you are searching a different url, you retry with this.

These links may be useful:

Is good practice paste the rpmlint output in the review, and at least make a build scratch on koji and paste the link in the review

Please tag the field "Blocks" with the word FE-NEEDSPONSOR. So, will let you know the sponsors you need sponsorship  

About the package 

I have not thoroughly checked the package, but at first glance,  I give some recommendations

You have a typo in this line
URL:		http://

in %install
make install prefix=${RPM_BUILD_ROOT}/usr
should be 
make install DESTDIR=%{buildroot} PREFIX=%{_prefix}

(if $DESTDIR supports)

Check it
$ rpm --eval %{_prefix}

the description should not be more than 80 characters per line, rpmlit complains with description-line-too-long

- Please do honors applicable compiler flags, Use %{optflags}

Check it
$ rpm --eval %{optflags}
-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4  -m64 -mtune=generic

Check the makefile and do the patch, (do not forget to send the patch to upstream)

        $(CC) $(CFLAGS) $(SRC)/bittwist.c -o $(SRC)/bittwist -I/usr/local/include -L/usr/local/lib -lpcap

        $(CC) $(CFLAGS) $(SRC)/bittwiste.c -o $(SRC)/bittwiste -I/usr/local/include -L/usr/local/lib -lpcap

- the tag license is wrong 
License:	GPL2
should be
License:        GPLv2

the changelog is wrong
* Fri Feb 15 2013 Giuseppe Marco Randazzo <gmrandazzo>
- first build for bittwist
should be
* Fri Feb 15 2013 Giuseppe Marco Randazzo <gmrandazzo> - 2.0-1
- first build for bittwist

- Don't needed %defattr(-,root,root)

Kind Regards

Comment 2 Fabian Affolter 2014-01-21 12:53:24 UTC
Giuseppe, are you still interested to go on with this package?

Comment 3 Fabian Affolter 2014-06-14 07:01:43 UTC

Comment 4 Giuseppe Marco Randazzo 2014-06-23 19:01:35 UTC
Hello, yes i am still interested.... sorry for the delay!! :-) Here all the modifications!! I hope now the spec file and the patch are fine!


Giuseppe Marco Randazzo

Comment 5 Giuseppe Marco Randazzo 2014-06-23 19:02:35 UTC
Created attachment 911551 [details]
bittwist new spec file

New Bitwist spec file

Comment 6 Giuseppe Marco Randazzo 2014-06-23 19:04:28 UTC
Created attachment 911552 [details]
bittwist-Makefile patch

The bittwist-Makefile patch.

~ Added the DESTDIR support
~ Fixed the make procedure to use the optflags

Comment 7 Fabian Affolter 2014-07-15 12:30:10 UTC
Can you please provide updated files (spec and SRPM)? Thanks.

Comment 8 Giuseppe Marco Randazzo 2014-07-15 12:42:36 UTC
Created attachment 918154 [details]
bittwist-2.0-2 SRC RPM  2014-07-15

SRC RPM bittwist 2.0-2

Comment 9 Giuseppe Marco Randazzo 2014-07-15 12:43:39 UTC
Created attachment 918155 [details]
bittwist spec file 2014-07-15

bittwist spec file 2014-07-15

Comment 10 Giuseppe Marco Randazzo 2014-07-15 12:44:35 UTC
Created attachment 918156 [details]
bittwist Make file patch 2014-07-15

bittwist 2.0-2 Make file patch 2014-07-15

Comment 11 Fabian Affolter 2015-03-04 09:48:40 UTC
Can you please provide updated files (spec and SRPM)? Not as attachment to the report but rather as links. Thanks.

Comment 12 Eduardo Echeverria 2015-03-05 06:18:20 UTC
I'm still interested in know what is the fas of Giuseppe and if still he hasn't been sponsored. Like says Fabian, Giuseppe, please append the links of the spec and SRPM, usually reviewers use a tool named fedora-review, that expects those links be available at the review.

Comment 13 Giuseppe Marco Randazzo 2020-04-28 15:57:07 UTC
Dear all,

thank you for your support and interest.
I found a workaround package, tcpreply.
With this message I would like to close this package request.

Best regards,

Comment 14 Cristian Ciupitu 2020-04-29 11:53:59 UTC
You mean tcpreplay

Note You need to log in before you can comment on or make changes to this bug.