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

Update latest to v4.0 of flightaware/dump1090 #42

Closed
e2jk opened this issue Feb 28, 2021 · 3 comments · Fixed by #48
Closed

Update latest to v4.0 of flightaware/dump1090 #42

e2jk opened this issue Feb 28, 2021 · 3 comments · Fixed by #48

Comments

@e2jk
Copy link
Contributor

e2jk commented Feb 28, 2021

Specifications

  • Image hash or version: latest/1.3.1
  • Platform: Raspberry Pi 1B (old ;) ), armv6
  • Subsystem: ?

Expected Behavior

I would expect the latest tag of this repo on the Docker Hub to use v4 of flightaware/dump1090.

Actual Behavior

The image currently pushed (one month ago) to the Docker Hub with tag latest is version 1.3.1 of this repo, which uses v3.8.1 of flightaware/dump1090.
There is an older (5 months ago) tagged flightaware-v4.0 which, based on its name, I'm assuming is using v4 of flightaware/dump1090.

Shouldn't latest point to v4?

Steps to Reproduce the Problem

  1. Install from Docker Hub targeting the latest tag
  2. Run and noticed that on the webpage it indicates v3.8.1 in the upper right of the page. I'd have expected v4.0
@Thom-x
Copy link
Owner

Thom-x commented Feb 28, 2021

Hello v4 has a bug #35, so I'm waiting for the fix (that is already on master, but no release yet...).

@e2jk
Copy link
Contributor Author

e2jk commented Feb 28, 2021

Merci @Thom-x.
I do see there are quite some changes on their dev branch (86 commits since v4.0), for now I'm compiling with v4.0 on my Raspi 1B to confirm flightaware/dump1090#115 is fixed in v4.0. Slow machine, but we'll get there at the end of day ;)

@Thom-x Thom-x linked a pull request Mar 23, 2021 that will close this issue
@Thom-x
Copy link
Owner

Thom-x commented Mar 23, 2021

fixed by #48

@Thom-x Thom-x closed this as completed Mar 23, 2021
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 a pull request may close this issue.

2 participants