Re: [mpd-devel] recent releases often stop to play

2014-03-14 Thread Max Kellermann
On 2014/03/14 16:12, Reindl Harald h.rei...@thelounge.net wrote: please let me know if there is any info i can provide to help debug Report a bug on the bug tracker. This is the developer mailing list which is for developers, not a user support mailing list.

Re: [mpd-devel] pull request: 'PulseOutputPlugin: support non-static output sinks'

2014-03-14 Thread Max Kellermann
On 2014/03/06 19:12, Pete Beardmore pete.beardm...@msn.com wrote: i've recently started using my secondary (internal) sound card for headphones output, and regularly move the mpd pulseaudio stream back and forth between the two outputs the issue i have/had is that when a playlist contains

Re: [mpd-devel] pull request: 'MultipleOutputs: ensure input_audio_format is zero-initialised'

2014-03-14 Thread Max Kellermann
On 2014/03/09 21:08, Pete Beardmore pete.beardm...@msn.com wrote: [branch: 'init.paused.fix'] hi, this single commit fixes the SIGABRT i experience when: 1. state file contains 'state: pause' 2. i start a new mpd instance Good catch, I've merged this. I changed this to be

Re: [mpd-devel] mpd: .wav file and ffmpeg 2.1

2014-03-17 Thread Max Kellermann
On 2014/03/17 16:09, Ricky Xian 2006...@qq.com wrote: Hi, all, As I try to use ffmpeg 2.1 with latest mpd 0.18.9, I found some issues below: 1.) seeking function doesn't work, I found that av_seek_frame function should be upgraded to avformat_seek_file, attached the patch. What

Re: [mpd-devel] add .mpdrootignore fallback giving recursive ignore

2014-05-13 Thread Max Kellermann
On 2014/04/17 23:12, Maxime Daniel l...@maxux.net wrote: Hi, Here is a patch I just made which add a .mpdrootignore on root directory. How does it works ? This file a like a .mpdignore but only on music root directory. This file is parsed and used only if a .mpdignore file is not found

Re: [mpd-devel] [PATCH] util: Fix header for strcasecmp

2014-08-15 Thread Max Kellermann
On 2014/08/15 21:22, François Revol re...@free.fr wrote: According to POSIX and both OSX and Linux manpages, strcasecmp comes from strings.h, not string.h. Most OSes also have them available in string.h, but we just fixed the headers on Haiku and it now only provides them in strings.h.

Re: [mpd-devel] mpc: Shell completion of files in subdirectories

2014-08-17 Thread Max Kellermann
On 2014/08/16 20:56, Kevin Locke ke...@kevinlocke.name wrote: If this seems like a reasonable solution, I would be happy to provide a patch. Yes, sounds reasonable. Max ___ mpd-devel mailing list mpd-devel@musicpd.org

Re: [mpd-devel] OHU/OHM SongCast Stream Decoder

2014-08-28 Thread Max Kellermann
On 2014/08/28 21:49, Craig Tolley ctolle...@hotmail.com wrote: Ok, not had any responses as yet I did reply to you. However, you only re-posted the same text until now. 1.I am creating an Input Plugin, how do I tell MPD that ohu:// and ohm:// are valid URIs? Somewhere I need to create

Re: [mpd-devel] Debug connection for Win32

2014-08-30 Thread Max Kellermann
On 2014/08/31 05:36, zw g pekingmas...@gmail.com wrote: Here is what i get from the command line after i set the log level to ' *verbose*' on Windows 7: [...] Please use the forum for user support requests. This mailing list is for developers only.

Re: [mpd-devel] Patches to support the MusicBrainz Release Track ID tag

2014-09-27 Thread Max Kellermann
On 2014/09/27 18:38, Wieland Hoffmann themi...@gmail.com wrote: the following two patches add support for the Release Track ID tag to MPD and libmpdclient. This tag is a unique identifier for a specific track on a release. The (similarly named) Track ID is not unique enough to do that since

Re: [mpd-devel] Unicode collation [Was: libicu]

2014-10-19 Thread Max Kellermann
On 2014/10/19 03:12, Ian Zimmerman i...@buug.org wrote: The version I have installed now is 0.17.6, and the problem is still present. MPD 0.17.6 has been out of support for a year. Now, would you please not send user support requests this mailing list? This is for developers only.

Re: [mpd-devel] [PATCH] Support S24_P32/S32/FLOAT sample formats on Pulse

2014-10-24 Thread Max Kellermann
On 2014/10/14 07:35, Ian Scott i...@polpo.org wrote: + ss.format = PA_SAMPLE_FLOAT32NE; [...] ss.format = PA_SAMPLE_S16NE; This cannot work. No matter what you write to ss.format, it will be overwritten right after the switch. But I like how simple this patch looks, compared

Re: [mpd-devel] [Patch] Database proxy plugin: retry connection attempts at startup

2014-11-30 Thread Max Kellermann
On 2014/11/30 19:52, Piotr Koz??owski pi...@pkozlowski.pl wrote: Hi, I've written a patch that allows configurable number of attempts to connect to remote MPD server if there are any errors. I'm running MPD proxy on a headless single board computer. It often happens that the first

Re: [mpd-devel] [Patch] Database proxy plugin: retry connection attempts at startup

2014-11-30 Thread Max Kellermann
On 2014/11/30 20:20, Piotr Koz??owski pi...@pkozlowski.pl wrote: Use of sleep() is unacceptable. Could you explain why? I've seen usage of usleep() in other parts of the code - would it be better here? Otherwise what do you suggest? I'm new to MPD development, so any advice is welcome. All

Re: [mpd-devel] systemd and zeroconf: No global port, disabling zeroconf

2014-12-02 Thread Max Kellermann
On 2014/12/03 02:56, Punky punky...@gmail.com wrote: In zeroconf module, listen_port variable is used. listen_port variable should be set to the value that MPD (successfully) listens to, and if TCP port (default 6600) is being configured But listen_port is ignored when systemd socket

Re: [mpd-devel] PATCH: Add support for configuration specified optional headers in HTTP output plugins

2014-12-03 Thread Max Kellermann
On 2014/12/03 22:00, X Ryl boite.pour.s...@gmail.com wrote: Please find attached a new patch, with your other remarks fixed. How is this change related to your patch description? -# encoder vorbis# optional, vorbis or lame +# encoder vorbis

Re: [mpd-devel] [PATCH 1/2] thread/Id.hxx: fix musl build error

2014-12-08 Thread Max Kellermann
On 2014/12/07 22:40, Jörg Krause jkra...@posteo.de wrote: This patch fix a build error with the musl C library. The patch is originally taken from Alpine Linux: http://git.alpinelinux.org/cgit/aports/tree/main/mpd/fix-musl.patch. musl declares pthread_equal both as a macro and as a function.

Re: [mpd-devel] [PATCH 1/2] thread/Id.hxx: fix musl build error

2014-12-08 Thread Max Kellermann
On 2014/12/09 00:10, Ben Boeckel maths...@gmail.com wrote: On Mon, Dec 08, 2014 at 22:14:16 +0100, Jörg Krause wrote: FYI: This is the statement of a musl maintainer about this issue: http://www.openwall.com/lists/musl/2014/12/08/15 That is???unfortunate. Macros are basically evil and with

Re: [mpd-devel] [PATCH 1/2] thread/Id.hxx: fix musl build error

2014-12-09 Thread Max Kellermann
On 2014/12/09 17:06, Ben Boeckel maths...@gmail.com wrote: On Tue, Dec 09, 2014 at 07:21:47 +0100, Max Kellermann wrote: This is C code, while the former is NOT C code - it escapes from the raw C language into the preprocessor language. My version is type-safe. And safe with C

Re: [mpd-devel] Tags from Input plugins

2014-12-14 Thread Max Kellermann
On 2014/12/15 00:14, Pizzocel pizzo...@canis.uberspace.de wrote: Hi, I'm trying to write an input plugin. It determines the content of the tag at the time the stream is opened. I thought, the tag should then be delivered as the result of the first call to ReadTag. Subsequent calls should

Re: [mpd-devel] Build error w/libavutil 52.3.0

2014-12-24 Thread Max Kellermann
On 2014/12/24 08:46, Jan Brittenson b...@rockgarden.net wrote: Fix: always include both libavutil/mem.h and libavcodec/avcodec.h regardless of whether we have av_fast_malloc or avcodec.h or not. But why include libavcodec/avcodec.h when none of its symbols is used?

Re: [mpd-devel] Build error w/libavutil 52.3.0

2014-12-24 Thread Max Kellermann
On 2014/12/24 22:13, Jan Brittenson b...@rockgarden.net wrote: But why include libavcodec/avcodec.h when none of its symbols is used? avcodec.h declares av_fast_malloc() and av_fast_realloc(). No. Only older versions do. Recent versions move these functions to libavutil. That's why

Re: [mpd-devel] [PATCH] Fixing the systemd service file against systemd205

2015-01-01 Thread Max Kellermann
On 2015/01/01 18:30, Reventlov cont...@volcanis.me wrote: Hello. Since Systemd 205, the low-level cgroup configuration options ControlGroup= and ControlGroupAttribute= have been removed. Relevant upstream changelog: http://lists.freedesktop.org/archives/systemd-devel/2013-July/011679.html

Re: [mpd-devel] Fwd: mpc diff - mpc git diff

2015-01-15 Thread Max Kellermann
On 2015/01/15 22:03, Alexander Hall alexan...@beard.se wrote: Fix use-after-free in mpc / cmd_seek() Merged, thanks. ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] [PATCHv4 2/2] settings: add support for reading ~/.authinfo file

2015-01-12 Thread Max Kellermann
On 2015/01/13 06:20, Avuton Olrich avu...@gmail.com wrote: Was this ever committed? No. I asked a question that was left unreplied, therefore I never returned to this patch. ___ mpd-devel mailing list mpd-devel@musicpd.org

Re: [mpd-devel] PATCH : Add option to set cache size of libcdio_paranoia

2015-01-14 Thread Max Kellermann
On 2014/12/28 18:05, Vincent Mussard vmussard.nos...@free.fr wrote: Dear all Here is a little patch that add an option cache_size to the cdio_paranoia plugin. This option set the libcdio_paranoia cache size (in sector), which is by default very high (1200 sectors == 16 seconds). On my

Re: [mpd-devel] mpd-website request pull

2015-02-10 Thread Max Kellermann
On 2015/02/09 12:59, kaliko kal...@azylum.org wrote: Hi, I've corrected a link for a python module page. Please consider merging the ref libs: Done ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] [PATCH] util: clarify that the error is from mpd

2015-02-13 Thread Max Kellermann
On 2015/02/08 02:39, Ben Boeckel maths...@gmail.com wrote: When mpc prints out 'error: ...', it is ambiguous where it came from. When the error comes from the mpd server, the user might be confused by something like: Merged ___ mpd-devel mailing list

Re: [mpd-devel] Can't control volume during SACD playback

2015-02-15 Thread Max Kellermann
On 2015/02/15 23:46, pisymbol . pisym...@gmail.com wrote: When using 'mixer_type=software' in my mpd.conf file, volume can not be controlled during DSF and SACD ISO playback (ISO support is via Maxim Anisiutkin's experimental plugin). First of all, this mailing list is for developers, not for

Re: [mpd-devel] [PATCH 2/2] TagStream: continue to scan if a plugin scanned a duration

2015-05-04 Thread Max Kellermann
On 2015/05/03 16:36, Thomas Guillem tho...@gllm.fr wrote: It happens when using mad and ffmpeg. The mad plugin will parse only the duration and ScanStream will return true. Therefore, ffmpeg plugin won't be used and no tags will be scanned. This would add a lot of overhead. What is the reason

Re: [mpd-devel] [PATCH 2/2] TagStream: continue to scan if a plugin scanned a duration

2015-05-04 Thread Max Kellermann
On 2015/05/04 15:55, Thomas Guillem tho...@gllm.fr wrote: I use mpd on android with libnfs (see https://github.com/tguillem/mpd/commits/android-2), therefore id3 is not available to scan files (because it uses a fd). Then change the ID3 code to use the stream API.

Re: [mpd-devel] [PATCH] Workaround: open() ignores 'mode' with O_TMPFILE

2015-05-21 Thread Max Kellermann
On 2015/05/21 09:02, Yclept Nemo orbisvi...@gmail.com wrote: And how do you know the bug is the same on the build machine as on the machine running MPD? Hmm, I didn't consider this. The problem is not that the build machine suffers the bug while distribution machines don't, but vice

Re: [mpd-devel] [PATCH] Workaround: open() ignores 'mode' with O_TMPFILE

2015-05-21 Thread Max Kellermann
On 2015/05/21 09:08, Yclept Nemo orbisvi...@gmail.com wrote: OK, the test case should be moved from configure.ac into the constructor of FileDescriptor. If you do this test every time a FileDescriptor gets constructed, what's the advantage of testing it in the first place?

Re: [mpd-devel] [PATCH 2/6] tag: Ape/Riff/Aiff/Id3 now use an InputStream

2015-06-22 Thread Max Kellermann
On 2015/06/05 16:27, Thomas Guillem tho...@gllm.fr wrote: This allow to use these scanners for remote InputStream like SMB and NFS. Fails to compile: test/dump_rva2.cxx: In function 'int main(int, char**)': dump_rva2.cxx:60:62: error: invalid initialization of non-const reference of type

Re: [mpd-devel] [PATCH 1/6] InputStream: add ReadFull method

2015-06-22 Thread Max Kellermann
On 2015/06/05 16:27, Thomas Guillem tho...@gllm.fr wrote: Convenient method that behave differently than Read, and that will be used by tag scanners. This method will return in case of error, if the whole data is read or is EOF is reached. Merged. Sorry for being so late.

Re: [mpd-devel] [PATCH] Emmit a warning if the OutputThread fails to get realtime scheduling

2015-06-22 Thread Max Kellermann
On 2015/06/18 12:02, Christian Halaszovich chalaszov...@gmail.com wrote: This only applies to linux systems. Here, sched_setscheduler() is called to get realtime scheduling. With this patch, the return value of this function is now checked and a warning and an error message are generated if it

Re: [mpd-devel] [PATCH 1/2] SetThreadRealtime() now reports if getting realtime scheduling was successful. (Only meaningful on Linux systems.)

2015-06-17 Thread Max Kellermann
On 2015/06/17 13:34, Christian Halaszovich chalaszov...@gmail.com wrote: Patch 1/2. Check the return value of sched_setscheduler() in SetThreadRealtime. Report to the caller, if we were successful. To this end SetThreadRealtime now returns false, if ached_setschedluer failed or true on all

Re: [mpd-devel] Add Elapsed time in mpc???

2015-06-16 Thread Max Kellermann
On 2015/06/12 09:29, salorium ??? salor...@hotmail.fr wrote: Hello, i want a git account for add elapsed time on mpc [..] Elapsed time: https://github.com/salorium/mpc But.. you already have a mpc git repository, why do you want another one? ___

Re: [mpd-devel] [PATCH] db/update/ExcludeList: implement with std::regex

2015-05-29 Thread Max Kellermann
On 2015/05/22 09:04, Thomas Guillem tho...@gllm.fr wrote: --- New version that use std::regex instead of regex.h. That is an incompatible change, and will break everybody's .mpdignore files! ___ mpd-devel mailing list mpd-devel@musicpd.org

Re: [mpd-devel] [Patch] Least unambiguous commands.

2015-05-29 Thread Max Kellermann
On 2014/11/12 15:19, Rory McNamara pink.banana.f...@gmail.com wrote: Good catch, I think I have them all now. Merged with a few more optimizations. ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] Add Elapsed time in mpc

2015-05-29 Thread Max Kellermann
On 2015/05/27 22:08, salorium ¢¾ salor...@hotmail.fr wrote: SSH public key(s) (as attachment or URL, do not copy and paste) : http://salorium.com/id_rsa.pubElapsed time : ERROR 404: Not Found. ___ mpd-devel mailing list mpd-devel@musicpd.org

Re: [mpd-devel] [PATCH] storage/nfs: fix deadlock when connecting

2015-05-29 Thread Max Kellermann
On 2015/05/09 15:59, Thomas Guillem tho...@gllm.fr wrote: The Connect method can be called between Schedule and lock. In that case, when locked, the state is already set to CONNECTING of READY and the condition won't be signaled anymore. Good catch. Merged to v0.19.x

Re: [mpd-devel] find track 1 not working as expected

2015-07-01 Thread Max Kellermann
On 2015/06/28 08:40, Andreas Mair amair@gmail.com wrote: I've taken a look at the MPD sources and I think the problem is, that MPD tries an exact *string* match. If I use ... find 01/10 it works, but I think that's not ideal to work like this because one has to know (1) how many songs an

Re: [mpd-devel] [PATCH 1/6] InputStream: add ReadFull method

2015-05-21 Thread Max Kellermann
On 2015/05/21 23:43, Thomas Guillem tho...@gllm.fr wrote: Convenient method that behave differently than Read, and that will be used by tag scanners. This method will return in case of error, if the whole data is read or is EOF is reached. But how will the caller know that an error has

Re: [mpd-devel] find track 1 not working as expected

2015-08-13 Thread Max Kellermann
On 2015/08/12 13:16, Andreas Mair amair@gmail.com wrote: Hi Max, OK, I see. I've changed the patch to cover your suggestion. Commit message is missing. ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] [PATCH] CueParser.cxx: ignore INDEX after first per track

2015-08-23 Thread Max Kellermann
On 2015/08/23 04:19, Will Tiffany will.tiff...@gmail.com wrote: Use the first INDEX in each TRACK section, instead of the last, for the start time. This preserves the original CD layout (including gaps between tracks), and avoids skipping sections of songs in more exotic cuesheets (eg musical

Re: [mpd-devel] [PATCH] Add support for ArtistSort and AlbumArtistSort tags

2015-08-20 Thread Max Kellermann
On 2015/08/19 22:26, Kim Tore Jensen k...@incendio.no wrote: I propose to add support for the ArtistSort and AlbumArtistSort tags in libmpdclient; see attached patch. These were left out of libmpdclient intentionally, because these tags appear to make sense only on the server side, for sorting.

Re: [mpd-devel] [PATCH] Add support for ArtistSort and AlbumArtistSort tags

2015-08-20 Thread Max Kellermann
On 2015/08/20 11:36, Kim Tore Jensen k...@incendio.no wrote: These tags are much needed in order to sort songs correctly at the client side, after having retrieved them from MPD using either a search or retrieving the library. I understand. If there was a way to sort search results

Re: [mpd-devel] [PATCH] Add support for ArtistSort and AlbumArtistSort tags

2015-08-20 Thread Max Kellermann
On 2015/08/20 13:21, Kim Tore Jensen k...@incendio.no wrote: Actually, yes. It'd be really useful to be able to manipulate search results locally in the client. For example: search within a search result with new sort parameters; re-sort a search result with new sort order; regular expression

Re: [mpd-devel] [PATCH] CurlInputPlugin : broken connectionsometimes --Solved patch

2015-08-11 Thread Max Kellermann
On 2015/08/11 16:19, Ben Boeckel maths...@gmail.com wrote: Maybe Max won't spell it out, but instead of attaching the patch, please use: git format-patch 34db35c36d8bb32b2c41640432474f52889cce08.. No, this is not a problem that can be solved with tools. The problem is that there is too

Re: [mpd-devel] [PATCH] MPC: command: let listall print songs only

2015-08-11 Thread Max Kellermann
On 2015/07/28 12:09, Thorsten Wißmann e...@thorsten-wissmann.de wrote: Formerly, the output of mpc listall --format '' included directory paths and not only songs. Attached you can find the patch 0001-command-let-listall-print-songs-only.patch which limits the output songs (as it is

Re: [mpd-devel] [PATCH 0/2] Some improvements for MPC display format

2015-08-11 Thread Max Kellermann
On 2015/08/09 17:23, Dmitri Vereshchagin dmitri.vereshcha...@gmail.com wrote: Hello, These patches add support for escape characters in format strings and setting format string through MPC_FORMAT environment variable. Merged ___ mpd-devel mailing

Re: [mpd-devel] CurlInputPlugin : broken connection sometimes -- Solved patch

2015-08-06 Thread Max Kellermann
On 2015/08/06 11:31, ?? maon...@qq.com wrote: Hi All, Attached is for solving this issue which I was discussing at my last email: http://mailman.blarg.de/pipermail/mpd-devel/2015-July/000375.html It seems work fine after I test, and anyone helps me to double check? Thanks. And

Re: [mpd-devel] [PATCH] mpd.service: Set the Documentation option

2015-08-06 Thread Max Kellermann
On 2015/07/17 19:55, Wieland Hoffmann themi...@gmail.com wrote: This makes references to mpd(1) and mpd(5) appear in systemd status output. Merged. ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] [PATCH] CurlInputPlugin : broken connection sometimes --Solved patch

2015-08-07 Thread Max Kellermann
On 2015/08/07 05:18, Alex maon...@qq.com wrote: Hi Max, I am so sorry that it's first time at here, so I don't know how to do that correctly. My patch was creating from mad-git, the reversion is 34db35c36d8bb32b2c41640432474f52889cce08. I don't understand that. So you're not the

Re: [mpd-devel] [PATCH] CurlInputPlugin : broken connectionsometimes --Solved patch

2015-08-07 Thread Max Kellermann
On 2015/08/07 08:50, Alex maon...@qq.com wrote: On 2015/08/07 05:18, Alex maon...@qq.com wrote: Hi Max, I am so sorry that it's first time at here, so I don't know how to do that correctly. My patch was creating from mad-git, the reversion is

Re: [mpd-devel] find track "1" not working as expected

2015-10-21 Thread Max Kellermann
On 2015/10/21 08:58, Andreas Mair wrote: > Hi Max, > > I've now created a pull request: Filter out this extra data and > leading zeroes in "track" and "disc" tags. Merged with a few minor corrections. By the way, please don't send GitHub pull requests. Those are very

Re: [mpd-devel] MPD fails to build if configured with "--disable-inotify"

2015-11-02 Thread Max Kellermann
On 2015/11/02 10:07, Andreas Mair wrote: > Hello, > > the subject says it all. http://bugs.musicpd.org/ ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] Digital filters handling in MPD protocol

2015-10-16 Thread Max Kellermann
On 2015/10/16 01:35, Aleksander Trofimowicz wrote: > 1. there is a room and willingness to accept such proposal It is impossible to judge your future proposal until one knows the proposal. I, for one, have no idea what this is about.

Re: [mpd-devel] Digital filters handling in MPD protocol

2015-10-16 Thread Max Kellermann
On 2015/10/16 11:29, Aleksander Trofimowicz wrote: > My aim is to create facilities for a basic filter management: list all > available filters (or filter sets) and select the current one. I think > it makes sense to include these to MPD protocol as digital filters > affect how

Re: [mpd-devel] Symmetic upsampling support (feature request)

2015-10-19 Thread Max Kellermann
On 2015/10/19 20:05, Fulco Giovanni wrote: > Depending on the quality of the DAC, to upsample the signal via software > might lead to better results. > About symmetrical upsampling, the aim is to reduce the computational effort > (assuming this might really be the case).

Re: [mpd-devel] State file lost on brutal shutdown, needs atomic writes

2015-08-30 Thread Max Kellermann
On 2015/08/28 19:28, Calimero calimerotek...@free.fr wrote: And with the rename() syscall, we replace the old file atomically with the new one. Look what git master is doing. ___ mpd-devel mailing list mpd-devel@musicpd.org

Re: [mpd-devel] [PATCH] Document API changes between each libmpdclient version

2015-09-17 Thread Max Kellermann
On 2015/08/27 13:27, Kim Tore Jensen wrote: > Hi, > > Attached is a patch which documents API changes between libmpdclient versions, > making it easier to add client support for old versions of > libmpdclient. Merged. ___ mpd-devel

Re: [mpd-devel] [PATCH] CueParser.cxx: Append pregap to previous track

2015-09-17 Thread Max Kellermann
On 2015/08/25 12:11, Will Tiffany wrote: > Only ignore indexes after first nonzero, preserving pregaps but > appending them to the previous track instead of prepending to the > current. The first index of the first track is used for the start time > regardless of its

Re: [mpd-devel] MPD port to Haiku

2015-09-17 Thread Max Kellermann
On 2015/09/17 22:18, François Revol wrote: > Here is an updated patchset for my MPD port to Haiku. > > First 4 patches fix the build. > > Next 5 patches add native resources to the binary. > > Last patch adds a native audio output, which mostly works > for what I've tested,

Re: [mpd-devel] [PATCH 2/2] docs: add vlc and mpv to the list of example applications

2015-12-15 Thread Max Kellermann
On 2015/10/27 03:40, Ben Boeckel wrote: > These are other popular clients. In particular, VLC is available on > mobile devices. Merged. ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] [PATCH 1/2] httpd: add a parameter for playing silence

2015-12-15 Thread Max Kellermann
On 2015/12/15 21:58, Ben Boeckel wrote: > Hmm. How about "play_silence_when_paused". Too long? "_on_pause"? No > "play_"? Maybe just "silence_paused"? That would give a hint on what it's about. It's hard to find the perfect name, but "play_silence" sounds bad to me because

Re: [mpd-devel] [PATCH 1/2] httpd: add a parameter for playing silence

2015-12-15 Thread Max Kellermann
On 2015/10/27 03:40, Ben Boeckel wrote: > Some clients act weird in the face of silence and buffer it rather thay > "playing" it. This causes severe lag when the stream is unpaused. There are other clients which time out when not getting any data. (All audio streaming

Re: [mpd-devel] mpd hangs, kill -9 necessary

2015-12-29 Thread Max Kellermann
On 2015/12/27 18:42, richard lucassen wrote: > Hello list, > > I run Debian Wheezy with native mpd-0.16.7-2. This has worked fine for > a few years as long as I do not choose a radiostream that is broken. In that > case > I need to -SIGKILL mpd. This afternoon I

Re: [mpd-devel] [PATCH] net/Resolver: correct includes for gettaddrinfo(3)

2016-06-12 Thread Max Kellermann
Both patches merged, thanks. ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] [PATCH] doc: Initial sndio documentation

2016-06-22 Thread Max Kellermann
On 2016/06/22 18:42, Dimitris Papastamos wrote: > From 8fb1e4514752efb4ba42a1d8595c7651c159f456 Mon Sep 17 00:00:00 2001 > From: Dimitris Papastamos > Date: Wed, 22 Jun 2016 17:41:06 +0100 > Subject: [PATCH] doc: Initial sndio documentation Merged

Re: [mpd-devel] [PATCH] sndio: Add option to select output device

2016-06-22 Thread Max Kellermann
On 2016/06/22 11:27, Dimitris Papastamos <s...@2f30.org> wrote: > On Wed, Jun 22, 2016 at 11:21:02AM +0200, Max Kellermann wrote: > > On 2016/06/22 11:10, Dimitris Papastamos <s...@2f30.org> wrote: > > > Must have been an oversight when I looked at other plugin code.

Re: [mpd-devel] [PATCH] sndio: Add option to select output device

2016-06-22 Thread Max Kellermann
On 2016/06/22 10:57, Dimitris Papastamos wrote: > >From 43cb9a089363301636317446765f7ca689d6b8c5 Mon Sep 17 00:00:00 2001 > From: sin > Date: Wed, 22 Jun 2016 09:41:54 +0100 > Subject: [PATCH] sndio: Add option to select output device Merged. I just noticed that

Re: [mpd-devel] [PATCH] sndio: Add option to select output device

2016-06-22 Thread Max Kellermann
On 2016/06/22 11:10, Dimitris Papastamos wrote: > Must have been an oversight when I looked at other plugin code. > The patch below removes the timer usage. Please rebase. I have conflicts. ___ mpd-devel mailing list

Re: [mpd-devel] Trying to set a tag to a song with mpd_song_feed

2016-06-22 Thread Max Kellermann
On 2016/06/22 13:00, Jörg Krause wrote: > I am using libmpdclient (version 2.10) to write a LuaJIT-based MPD > client. I run into an issue when I am trying to add a comment to a > radio station URI which I add to the queue. The comment is successfully > feed to the

Re: [mpd-devel] [PATCH] configure.ac: check if libatomic is needed

2016-02-09 Thread Max Kellermann
On 2016/02/07 22:11, Thomas Petazzoni wrote: > The mpd source code uses the C++11 functionality, which > internally is implemented using the __atomic_*() gcc built-ins. On > certain architectures, the __atomic_*() built-ins are implemented in > the libatomic

Re: [mpd-devel] idle stored_playlist does not notice filesystem changes

2016-02-13 Thread Max Kellermann
On 2016/02/12 18:34, Gavin Kistner wrote: > Using the "idle stored_playlist" command works fine when MPD is used to > modify a playlist file. However, idle does not notice if the playlist file is > edited by a non-MPD source. > > Is there a .conf setting (similar to

Re: [mpd-devel] Pull request to support TAK files

2016-02-19 Thread Max Kellermann
On 2016/02/13 06:15, Chris Spiegel wrote: > In my git repository: > > https://github.com/cspiegel/mpd > > I've created a branch called tak which adds support for the TAK file > format (http://www.thbeck.de/Tak/Tak.html) to the FFmpeg plugin. FFmpeg > already has TAK

Re: [mpd-devel] mpd creates two ALBUMARTIST entries

2016-04-19 Thread Max Kellermann
On 2016/04/19 17:35, Julien Cubizolles wrote: > I noticed that music files metadata contain both "ALBUM ARTIST" and > "ALBUMARTIST" fields. When both are set, mpd creates ALBUMARTIST fields > in the tag_cache file. That's not a problem for mpd itself but emms gets > confused

Re: [mpd-devel] mpd creates two ALBUMARTIST entries

2016-04-19 Thread Max Kellermann
On 2016/04/19 17:51, Julien Cubizolles wrote: > I think mpd shouldn't create two ALBUMARTIST entries in its cache for > the same file. And I think you shouldn't create files with two ALBUMARTIST entries. Shit in, shit out. ___

Re: [mpd-devel] mpd takes 1.5s to execute previous/next while playing (regression)

2016-04-16 Thread Max Kellermann
On 2016/04/16 00:58, Calimero wrote: > I believe I have discovered an regression where mpd's response times on the > control socket got slow while playing. > `mpc next`, `mpc previous` or `telnet localhost 6600` and typing next or > previous can take a whole second and a

Re: [mpd-devel] [PATCH] input/alsa: prevent cpu hogging in nonblocking mode

2017-02-24 Thread Max Kellermann
On 2017/02/23 21:44, Joerg Raftopoulos wrote: > MPD's ALSA input plugin works since > 791efc171a35848bb062aad0aee41220cd1f62ba in SND_PCM_NONBLOCK mode. > According to ALSA documentation >

Re: [mpd-devel] [PATCH] fix build on i686 (config.h import)

2017-02-13 Thread Max Kellermann
On 2017/02/13 13:16, Profpatsch wrote: > It doesn???t build on i686 That's what the other guy said as well, but what does that mean? "It doesn't build" or "it doesn't work" are the worst problem descriptions I can imagine. No information. I can't understand the problem

Re: [mpd-devel] [PATCH 1/1] Fix for priority order bug if reordering in SetRandom().

2016-10-25 Thread Max Kellermann
On 2016/10/01 06:39, Eugene Baklanov wrote: > Fix for the problem where order with priorities gets out of whack in case it's > reordered by SetRandom() while another song is currently playing. > What happens is, if some song is already playing and you have set some >

Re: [mpd-devel] sidplay plugin (libsidplayfp) load rom images

2017-01-02 Thread Max Kellermann
On 2017/01/03 00:32, Jörg Raftopoulos wrote: > No HTML, please. ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] sidplay plugin (libsidplayfp) load rom images

2017-01-03 Thread Max Kellermann
On 2017/01/03 10:55, Jörg Raftopoulos wrote: > Darn webmailer. Patch attached as a zip file. But... what is it? No description, no documentation. Oh, and please don't use iostreams. It's a bloated mess. ___ mpd-devel mailing

Re: [mpd-devel] sidplay plugin (libsidplayfp) load rom images

2017-01-09 Thread Max Kellermann
On 2017/01/06 21:03, Jörg Raftopoulos wrote: > Now without fstream as requested. There is still no patch description (= commit message). Did you ensure that your code compiles with the old libsidplay? It looks suspicious. > + buffer = new char[romSize];

Re: [mpd-devel] [PATCH] Fix for priority order bug if reordering in SetRandom()

2017-01-04 Thread Max Kellermann
On 2016/11/12 02:11, Eugene Baklanov wrote: > Resending the patch after I screwed up with git patch formatting. Merged. ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] [PATCH] lyrics: lyrics area regex fix for LyricWikia

2017-01-03 Thread Max Kellermann
On 2016/11/10 18:04, lostd wrote: > From: Lazaros Koromilas Merged ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

Re: [mpd-devel] sidplay plugin (libsidplayfp) load rom images

2017-01-07 Thread Max Kellermann
On 2017/01/06 22:37, Jacob Thomas Errington wrote: > Shouldn't this be spelled "kernel" not "kernal"? No, it's really "kernal" on C64. I found this one suspicious, too, and looked it up. ___ mpd-devel mailing list

Re: [mpd-devel] [PATCH] ReusableArray: fix build error on GCC7

2017-03-01 Thread Max Kellermann
On 2017/02/27 03:34, Ben Boeckel wrote: > GCC7 outputs the following error without this change: > > src/util/ReusableArray.hxx:61:35: error: no matching function for call to > ???swap(size_t&, const size_t&)??? >std::swap(capacity, src.capacity); > > which can

Re: [mpd-devel] Was this change reverted by accident?

2017-03-01 Thread Max Kellermann
On 2017/02/27 17:22, Eric Wollesen wrote: > It was merged, but then immediately it seems to have been reverted in this > commit: > > 021519f command/QueueCommands: eliminate duplicate code in > handle_addid() It was not reverted - look what happens after these lines, now

Re: [mpd-devel] Receive command from Named Pipe and treat it as a client PlayerCommand

2017-08-11 Thread Max Kellermann
On 2017/08/10 17:35, ed mcmurray wrote: > From within the MPD application, I would like to access a Linux Named Pipe > FIFO that has been created in a second application. The second application > would place a STOP command into the Name Pipe, when appropriate. I would >

Re: [mpd-devel] Receive command from Named Pipe and treat it as a client PlayerCommand

2017-08-11 Thread Max Kellermann
On 2017/08/11 20:30, Kurt Van Dijck wrote: > Even more nitpicking: > mkfifo testfifo > echo hello world > testfifo > ... wait ... > cat testfifo > now echo has completed. This has nothing to do with buffering or writes, because a write hasn't happened at

Re: [mpd-devel] Receive command from Named Pipe and treat it as a client PlayerCommand

2017-08-11 Thread Max Kellermann
On 2017/08/11 13:54, Kurt Van Dijck wrote: > in order to successfully write() to a fifo, the read() must > complete. Nitpicking: that part is not true. Linux has a pipe buffer as well, which defaults to 64 kB (configurable). Maybe other kernels don't have one,

Re: [mpd-devel] Receive command from Named Pipe and treat it as a client PlayerCommand

2017-08-11 Thread Max Kellermann
On 2017/08/11 13:45, ed mcmurray wrote: > On the MPD side I'm not so sure where in the code that I should create, > bind, listen and accept connections, for the socket. That code already exists. Why do you believe a change to MPD is necessary? > I'm sure this should be

Re: [mpd-devel] decoder/ffmpeg: add support for adx

2017-06-01 Thread Max Kellermann
On 2017/06/01 19:16, Jörg Raftopoulos wrote: > Add ffmpeg decoder support for *.adx files (Sega game console) Your patch contains very strange Unicode characters instead of tabs. git can't apply that. ___ mpd-devel mailing list

Re: [mpd-devel] MPD distribution.

2017-11-27 Thread Max Kellermann
mail, because our download provider CDN received a > request from email address "m...@duempel.org" with Max Kellermann reqesting > to remove image from their server, as we somehow violate GPL licensing > rules of MPD. > > We think we do not break any GPL rules, as we just offe

Re: [mpd-devel] Formal request for GPL source code of Eupony

2017-11-23 Thread Max Kellermann
enforcement lawyer to your case. Regards, Max Kellermann On 2017/10/25 08:33, Max Kellermann <m...@duempel.org> wrote: > Dead sir or madam, > > I just downloaded from your website: > > http://euphonyimage-798b.kxcdn.com/euphony2017

[mpd-devel] Request for GPL source code of your Aurender A10 product

2017-12-17 Thread Max Kellermann
ing all of your products containing my copyrighted software illegal. I expect to receive the full source code and full GPL compliance within two seeks, that is until December 31st 2017. Regards, Max Kellermann ___ mpd-devel mailing list mpd-devel@musicpd.org http://mailman.blarg.de/listinfo/mpd-devel

  1   2   >