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

Sort by Composer #612

Open
p5a0u9l opened this issue Jan 19, 2017 · 5 comments
Open

Sort by Composer #612

p5a0u9l opened this issue Jan 19, 2017 · 5 comments

Comments

@p5a0u9l
Copy link

p5a0u9l commented Jan 19, 2017

#How reasonable is it to request a feature that allows better support for music with extended tags, most importantly, Composer? One in particular would be extending aaa_mode to aaac_mode (which of course breaks the aaa symmetry, but you see the idea hopefully).

If similar requests (support for classical music tags, etc.) has previously come up in forums, pardon and point me there.

To sweeten the deal, I'd be happy to contribute said support where possible. I'm familiar with c, but less so with contributing to large open-source projects, but have been looking for a good opportunity to dive in somewhere.

EDIT: not sure if submitters can label these as "Feature Requests"?

@nefthy
Copy link
Collaborator

nefthy commented Jan 19, 2017

To sweeten the deal, I'd be happy to contribute said support where possible. I'm familiar with c, but less so with contributing to large open-source projects, but have been looking for a good opportunity to dive in somewhere.

cmus is quite welcoming contributions. Just clone the repository, hack away and make a pull request when your done. However keep the following in mind:

  • Use and keep the formatting (linux kernel convetions) and style (c99), that is used in the source files.
  • Don't break current behavior if possible. If this is not the case, discuss it with the maintainers (@flyingmutant , @mahkoh) first.

@mahkoh
Copy link
Member

mahkoh commented Jan 21, 2017

Before writing any code you should explain in more detail what changes you want.

@p5a0u9l
Copy link
Author

p5a0u9l commented Jan 22, 2017

@mahkoh agreed completely. If I can communicate the functionality I'm after sufficiently, you will be
better equipped to point me to the relevant places in the code as well as advise on the feasibility of the changes.

The general theme is increased support for classical music tags in cmus. Here's a simple proposed starting point,

Add the "composer" tag to the list of Filter strings, which, from the man page, currently are limited
to
filename, artist, albumartist, album, title, genre, comment, codec, codec_profile, media

this would enable creating a filter like,
composer="Bach"&artist="*Perahia"

More ambitious functionality would add Composer to view 1 in addition to Artist/Album. Basically, allow sort by Composer OR Artist OR Album. But, how's starting with the first idea, where in the code should I look? Cheers

@paulrbts
Copy link

paulrbts commented Mar 8, 2017

I'd definitely support/use this! I don't know C so can't help write it but if I can help test it, I'd gladly help.

@astralspurs
Copy link

I'd also like to emphasize this feature request. Searching by composer, performer, record label, and language would be useful on a daily basis to me. All of these fields mentioned are standard fields in id3v2 and flac vorbis comments. Right now I have to store so much of this information in the comment field to have cmus have access to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants