On 2012/08/08 17:48, Jurgen Kramer <gtmkra...@xs4all.nl> wrote: > On Tue, 2012-08-07 at 20:02 +0200, Max Kellermann wrote: > > On 2012/08/03 15:22, Jurgen Kramer <gtmkra...@xs4all.nl> wrote: > > > Commit: > > > http://git.musicpd.org/cgit/gtmkramer/mpd.git/commit/?id=0156b6b1717bfc3c43fee6a6a6c6249f2abcb753 > > > > > > Repo: > > > http://git.musicpd.org/cgit/gtmkramer/mpd.git/ > > > > Fails to build when libid3tag is disabled: > > > > src/decoder/dsdlib.c:142: error: undefined reference to 'id3_tag_parse' > > src/decoder/dsdlib.c:146: error: undefined reference to 'tag_id3_import' > Thanks, fixed it
You added a new commit that is supposed to fix the bug in the first commit, but I do not want to merge the first commit when we already know it's faulty, because it will make bisecting difficult. These two commits should be folded into one before I merge the feature. > I am still not completely comfortable with git, hopefully correct now. It's not. You did not rebase, you merged again. Rebasing is as easy as typing: git pull --rebase git://git.musicpd.org/master/mpd.git master ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ Musicpd-dev-team mailing list Musicpd-dev-team@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team