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

malformed links when using the command line #348

Closed
ahmadnassri opened this issue Jul 8, 2018 · 7 comments
Closed

malformed links when using the command line #348

ahmadnassri opened this issue Jul 8, 2018 · 7 comments

Comments

@ahmadnassri
Copy link

@ahmadnassri ahmadnassri commented Jul 8, 2018

e.g.

LINK="https://hacker.bio/hackers/dan-shaw/"
mid3v2 -e \
  --WOAF "$LINK" \
  --WOAS "$LINK" \
  --WORS "https://hacker.bio/" \
  --WPUB "https://techmasters.community/" \
  --WXXX "$LINK" \
audio.mp3

result:

WFED=[u'https://hacker.bio/feed.xml']
WOAF=[u'https://hacker.bio/hackers/dan-shaw/']
WOAS=[u'https://hacker.bio/hackers/dan-shaw/']
WORS=[u'https://hacker.bio/']
WPUB=[u'https://techmasters.community/']
WXXX=[u'https://hacker.bio/hackers/dan-shaw/']

expected result:

WFED=https://hacker.bio/feed.xml
WOAF=https://hacker.bio/hackers/dan-shaw/
WOAS=https://hacker.bio/hackers/dan-shaw/
WORS=https://hacker.bio/
WPUB=https://techmasters.community/
WXXX=https://hacker.bio/hackers/dan-shaw/
@lazka

This comment has been minimized.

Copy link
Member

@lazka lazka commented Jul 8, 2018

What command did you use to create the "result"?

@ahmadnassri

This comment has been minimized.

Copy link
Author

@ahmadnassri ahmadnassri commented Jul 8, 2018

I'm only showing the relevant portion from:

mid3v2 --list audio.mp3

also FYI:

$ mid3v2 --version
mid3v2 1.3
Uses Mutagen 1.38
@lazka

This comment has been minimized.

Copy link
Member

@lazka lazka commented Jul 8, 2018

ah, right, this is #344

I'll try to get a new release out.

@ahmadnassri

This comment has been minimized.

Copy link
Author

@ahmadnassri ahmadnassri commented Jul 8, 2018

👍 thanks

@ahmadnassri

This comment has been minimized.

Copy link
Author

@ahmadnassri ahmadnassri commented Jul 8, 2018

so I installed using pip and I still get the same result:

mid3v2 1.3
Uses Mutagen 1.40.0
@ahmadnassri

This comment has been minimized.

Copy link
Author

@ahmadnassri ahmadnassri commented Jul 8, 2018

mediainfo audio.mp3

also shows the same issue with the format of the values:

Publisher/Url                            : [u'https://techmasters.community/']
Official audio file webpage              : [u'https://hacker.bio/hackers/dan-shaw/']
Official audio source webpage            : [u'https://hacker.bio/hackers/dan-shaw/']
URL                                      : [u'https://hacker.bio/hackers/dan-shaw/']
@lazka

This comment has been minimized.

Copy link
Member

@lazka lazka commented Jul 16, 2018

New release is out.

Thanks for your report!

@lazka lazka closed this Jul 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.