Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add termshark #3931

Closed
wants to merge 1 commit into from
Closed

Add termshark #3931

wants to merge 1 commit into from

Conversation

gcla
Copy link
Contributor

@gcla gcla commented Jun 8, 2019

Termshark is a terminal UI for tshark. It's written in golang and as such
consists of one binary. This is a minimal termux build script to make
termshark available. Termshark depends on tshark at runtime, so tshark (from
wireshark) is a package dependency.

More info is available at https://github.com/gcla/termshark.

Termshark is a terminal UI for tshark. It's written in golang and as such
consists of one binary. This is a minimal termux build script to make
termshark available. Termshark depends on tshark at runtime, so tshark (from
wireshark) is a package dependency.

More info is available at https://github.com/gcla/termshark.
@Grimler91
Copy link
Member

Cool! We don't have a package named wireshark though, but wireshark-gtk | tshark should do it (so that it requires either the gtk variant from x11-packages or tshark from termux-root-packges)

@ghost
Copy link

ghost commented Jun 8, 2019

Dependencies must be present in repository. This PR should be opened in https://github.com/termux/termux-root-packages where tshark is available.

@gcla
Copy link
Contributor Author

gcla commented Jun 8, 2019

Apologies, I'll re-submit the PR there and withdraw this one.

@gcla
Copy link
Contributor Author

gcla commented Jun 8, 2019

Hi - I misunderstood something, and would love to get some advice :-)

Reading the docs, termux-root-packages is for users with rooted phones. I could submit termshark there, but termshark itself doesn't require a rooted phone, and indeed is probably more useful for inspection of pcap files e.g. download a pcap from somewhere onto your phone, then run termshark in termux to look at it (like wireshark). Or, I could submit against x11-packages and depend on wireshark-gtk, although termshark itself doesn't require X11 to run, just the tshark binary.

I tested termshark on termux by installing x11-packages and running the termshark binary to make sure it worked correctly. So maybe the best option would be to move the PR to x11-packages, and that would make it available to the most users?

Thanks!

@Grimler91
Copy link
Member

Grimler91 commented Jun 8, 2019

@xeffyr I think wireshark-gtk | tshark makes most sense as wireshark-gtk also provides tshark. Cirrus-CI will still be able to build the package since ./build-package.sh -i can download from the other repos as well.

And if it depends on either wireshark-gtk or tshark then having it in termux-packages makes sense.

edit: No, sorry, ./build-package.sh -i won't work unless it knows which version to download, termux-root-packages might make the most sense then

@gcla
Copy link
Contributor Author

gcla commented Jun 9, 2019

Hi @xeffyr and @Grimler91 - thanks for the advice. I understand now that even though packages in termux-root-packages are expected to require a rooted phone, as a non-rooted-phone user, I can still enable the repo and install from that source if I want. So putting termshark in this package source won't limit its availability to a smaller subset of termux users. I'll move the PR over there!

gcla added a commit to gcla/termux-root-packages that referenced this pull request Jun 9, 2019
Termshark is a terminal UI for tshark. It's written in golang and as such
consists of one binary. This is a minimal termux build script to make
termshark available.

I originally made this PR at termux-packages -
termux/termux-packages#3931 - but the recommendation
from that discussion was to submit against root packages instead. Termshark
needs tshark at runtime, and tshark could come from either
x11-repo:wireshark-gtk or root-repo:tshark.

More info on termshark is available at https://github.com/gcla/termshark.
@gcla gcla closed this Jun 9, 2019
ghost pushed a commit to termux/termux-root-packages that referenced this pull request Jun 9, 2019
Termshark is a terminal UI for tshark. It's written in golang and as such
consists of one binary. This is a minimal termux build script to make
termshark available.

I originally made this PR at termux-packages -
termux/termux-packages#3931 - but the recommendation
from that discussion was to submit against root packages instead. Termshark
needs tshark at runtime, and tshark could come from either
x11-repo:wireshark-gtk or root-repo:tshark.

More info on termshark is available at https://github.com/gcla/termshark.
ghost pushed a commit to termux/termux-root-packages that referenced this pull request Jun 9, 2019
Termshark is a terminal UI for tshark. It's written in golang and as such
consists of one binary. This is a minimal termux build script to make
termshark available.

I originally made this PR at termux-packages -
termux/termux-packages#3931 - but the recommendation
from that discussion was to submit against root packages instead. Termshark
needs tshark at runtime, and tshark could come from either
x11-repo:wireshark-gtk or root-repo:tshark.

More info on termshark is available at https://github.com/gcla/termshark.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants