ubuntuusers.de

Hi!

Mir ist noch ein kleiner Fehler in den Save-Song-Patches aufgefallen, der sich wohl bei der Weiterentwicklung wieder eingeschlichen hat: Bei fehlendem Albumnamen wird nicht, wie eigentlich vorgesehen, "unknown" statt des Albumnamens eingesetzt; die Variable "verschwindet" (es wird also beispielsweise kein Verzeichnis "artist/unknown/titel" angelegt, sonder nur artist/titel).

Um das wieder hinzubekommen, muss im patch (sowohl intrepid als auch hardy) in Zeile 517 nach album ein

[0]

eingefügt werden, es muss also statt

+                if (album == '\0') {

+                if (album[0] == '\0') {

verwendet werden (in controller.c dürfte es ca. Zeile 1165 sein).

Ich hatte zebob gebeten, das mit aufzunehmen; in den letzten veröffentlichten Versionen (hier im Thread und bei mtron auf der Homepage) fehlt es aber noch.

Auch bei den id3-tags wird nichts eingetragen, was aber weniger "dramatisch" ist; wer dort bei Album [unknown] stehen haben möchte, kann in vagalume/src/contoller.c (Zeilen 1110 ff. Sektion controller_set_tags (GstElement *element)) statt

        gst_tag_list_add (taglist, GST_TAG_MERGE_REPLACE_ALL, GST_TAG_TITLE, 
                t->title, GST_TAG_ARTIST, t->artist, GST_TAG_ALBUM, t->album, NULL);

         if (t->album[0] == '\0') {
         gst_tag_list_add (taglist, GST_TAG_MERGE_REPLACE_ALL, GST_TAG_TITLE, 
                 t->title, GST_TAG_ARTIST, t->artist, GST_TAG_ALBUM, "[unknown]",  NULL);
         } else { gst_tag_list_add (taglist, GST_TAG_MERGE_REPLACE_ALL, GST_TAG_TITLE, 
                 t->title, GST_TAG_ARTIST, t->artist, GST_TAG_ALBUM, t->album,  NULL);
         }

verwenden. Is' aber eher 'ne Kleinigkeit; möchte dafür eigentlich keinen eigenen patch bauen...

so long
hank

Hallo hank!

Danke für den Bugreport. ich hab den patch gefixt und deine Änderungen eingebaut ^[1]. Die Versionsnummer hab ich bei v9 belassen (das wird sozusagen die bugfix release 😉

Bez. intrepid pakete sind wir mittlerweile bei vagalume_0.7-1~intrepid5_i386.deb ^[2]. Mit letztem save-song und volume-bar patch. Das wird höchst wahrscheinlich die "stable" für intrepid, ausser es taucht noch was auf.

Zur wikiseite: Eventuell würde ich es so machen dass man einen generellen artikel über das originale vagalume schreibt (der möglichst generisch ist und nicht mit jeder neuen Release von Vagalume upgedatet werden muss). Einen eigenen artikel der sich ausschliesslich mit dem patchen und den neuen funktionen befasst, könnte man ja dann up-to-date halten.

cheers, mtron

^[1] http://mtrons.googlepages.com/vagamule-changelog.txt

^[2] http://mtrons.googlepages.com/vagalume_0.7-1~intrepid5_i386.deb

Hi!

mtron schrieb:

Danke für den Bugreport. ich hab den patch gefixt und deine Änderungen eingebaut ^[1]. Die Versionsnummer hab ich bei v9 belassen (das wird sozusagen die bugfix release 😉

Bez. intrepid pakete sind wir mittlerweile bei vagalume_0.7-1~intrepid5_i386.deb ^[2]. Mit letztem save-song und volume-bar patch. Das wird höchst wahrscheinlich die "stable" für intrepid, ausser es taucht noch was auf.

Hm, in die patches, die ich gerade von deiner Seite heruntergeladen habe, sind aber die Änderung nicht eingeflossen - hast du die nur für das "neuste" .deb verwendet?

vagamule ist ja auch schön - ist das jetzt der aktuelle Name für die inoffizielle Version? 😀 Und noch was: wir sollten uns bei den zusätzlichen patches auf einen Namen einigen - im Wiki steht z.B. 03_multiple-usage.patch, bei dir auf der Seite heißt er 03_multi-session.patch; mit dem neuen Namen an sich hab' ich kein Problem, aber verschiedenen verwirren nur 😐

Zum Wiki-Artikel ->bitte da weiterdiskutieren (ist extra getrennt worden...)

so long
hank

EDIT Sehe gerade, dass die Veränderungen in dem .tar.gz-Archiv gelandet sind; willst du sie auch in die anderen "Einzel-Patches" auf deiner Seite mit aufnehmen? Wäre für den hardy-patch ganz sinnig; sonnst kann ich auch einen bauen und aus dem Wiki hierher verlinken.

huch, hab ich doch glatt, vergessen die patches auf der hp zu verlinken... Ist jetzt geschehen.

Kannst du (oder sonstwer) bitte den save-song-v9_hardy.patch testen? ich hab hier leider keine möglichkeit dazu, dementsprechend ist das hardy binary noch auf der alten Version. Wär schön, wenn mir jemand eines per PN zukommen lassen könnte.

Bitte nicht hier anhängen sonst streikt wieder das forum ☺

Intrepid Version ist am letzten Stand.

cheers, mtron

Hi!

Wenn du nichts anderes verändert hast kann ich nur sagen, dass er funktioniert...

Anhängen ist kein Problem, wenn nicht gleich zwei Anhänge verwendet werden... 😉 ; per PN kann man leider nix mehr per Anhang verschicken.

Hänge mal mein aktuelles .deb mit volume-bar und multi-session an; hab's mit deinem intrepid-tar.gz. gebaut (natürlich mit dem hardy-patch mit den bugfixes, nur das changelog geändert).

so long
hank

Herzlichen Dank, many thanks an mtron, Heinrich Schwietering und zebob für die Volume-Bar und alles andere, was nun Vagalume ausmacht!!

@Robert-Andre Mauchin: vagalume is now a dream for me!! Thank you - Merci beaucoup!!

Grek336 schrieb:

Im Moment versuche ich eine lauffähige Version für mein Gutsy Gibbon zu erstellen (Save Song + Volume Bar). Leider gibt es da noch Probleme.

Das wird nix. Letzte Version des save-song patches mit unterstützung für gutsy war v5. Grund ist dass danach gio für dateioperationen verwendet wird, was gutsy mit der mitgelieferten gnome version nicht unterstützt. Also könnte es für gutsy mit vagalume-0.7 ^[1], save-song-v5 ^[2] & dem volumebar patch ^[3] funktionieren. Den volume Bar patch musst du aber auf die geänderten Sourcen anpassen der wurde für eine neuere Version des save-song patches gemacht. Vielleicht hast du mehr glück mit zebob's original "volfix" commit auf garage:vagalumne.^[4]

Mein Versuch die gepatchte Version von mtron ohne die Patches unter Gutsy zu kompilieren führt zum selben Fehler. Mit der Debian-Version http://packages.debian.org/sid/vagalume gibt es jedoch keine Probleme. Irgendwie muss sich ein Fehler eingeschlichen haben. Ist die Version von mtron aus der Debian-Version oder direkt aus dem Original-Quellcode abgeleitet?

.orig ist immer die original version (=upstream source) vom Author (Berto) Ob du sie von mir, dem ubuntu / debian repositories oder von Berto direkt holst ist egal. Es ist immer derselbe Code. Alle änderungen am original werden über externe patches gespeichert. und je nach der konfiguration in der debian directory unterscheiden sich die pakete der verschiedenen distributoren.

^[1] http://people.igalia.com/berto/files/vagalume/source/vagalume_0.7.orig.tar.gz

^[2] http://mtrons.googlepages.com/save-song-v5.patch

^[3] http://media.ubuntuusers.de/forum/attachments/1612872/02_volume-bar2.patch

^[4] https://garage.maemo.org/tracker/download.php/447/1709/2881/327/volfix.patch

Hallo,

ich hab jetzt mal versucht, das gepatchte Vagalume unter Gentoo zum laufen zu bringen... ohne Erfolg. Ich habe sowohl den Intrepid- als auch den Hardy- Sava-Song-Patch--9 versucht, immer kommt die Meldung:

../libtool: line 827: X--tag=CC: command not found
../libtool: line 860: libtool: ignoring unknown tag : command not found
../libtool: line 827: X--mode=link: command not found
../libtool: line 994: *** Warning: inferring the mode of operation is deprecated.: command not found
../libtool: line 995: *** Future versions of Libtool will require --mode=MODE be specified.: command not found
../libtool: line 2239: X-O2: command not found
../libtool: line 2074: X-march=i686: command not found
../libtool: line 2239: X-pipe: command not found
../libtool: line 2202: X-Wl,-O1: command not found

Fatal server error:
Server is already active for display 0
	If this server is no longer running, remove /tmp/.X0-lock
	and start again.

../libtool: line 2408: Xvagalume: command not found

Fatal server error:
Server is already active for display 0
	If this server is no longer running, remove /tmp/.X0-lock
	and start again.

../libtool: line 2420: Xvagalume: command not found
../libtool: line 2428: mkdir /.libs: No such file or directory
ACCESS DENIED  mkdir:     /.libs
mkdir: cannot create directory `/.libs': Permission denied
make[2]: *** [vagalume] Fehler 1
make[2]: Leaving directory `/var/tmp/portage/media-sound/vagalume-0.7/work/vagalume-0.7.orig/src'
make[1]: *** [all-recursive] Fehler 1
make[1]: Leaving directory `/var/tmp/portage/media-sound/vagalume-0.7/work/vagalume-0.7.orig'
make: *** [all] Fehler 2
 * 
 * ERROR: media-sound/vagalume-0.7 failed.
 * Call stack:
 *               ebuild.sh, line   49:  Called src_compile
 *             environment, line 2580:  Called gnome2_src_compile
 *             environment, line 1969:  Called die
 * The specific snippet of code:
 *       emake || die "compile failure"
 *  The die message:
 *   compile failure
 * 
 * If you need support, post the topmost build error, and the call stack if relevant.
 * A complete build log is located at '/var/tmp/portage/media-sound/vagalume-0.7/temp/build.log'.
 * The ebuild environment file is located at '/var/tmp/portage/media-sound/vagalume-0.7/temp/environment'.
 * 
--------------------------- ACCESS VIOLATION SUMMARY ---------------------------
LOG FILE = "/var/log/sandbox/sandbox-24568.log"

mkdir:     /.libs
--------------------------------------------------------------------------------

Liegt das an der libtool-version? Momentan ist 2.2.6a installiert.

Der "alte" Savesong -Patch, der die Titel "nur" herunterlädt, funktioniert übrigens.

(oO ich weiss dass das kein Gentoo-, sondern ein Ubuntuforum ist, aber der Patch ist nunmal hier am bekanntesten, denke ich. Vll weiß jemand Rat)

edit:

Wenn ich das nicht als root, sondern als sudo kompiliere, kommen noch mehr Fehlermeldungen:

http://paste.ubuntuusers.de/392403/

...

Hi!

Ich habe noch ein wenig an dem 03_multiple-usage.patch gefeilt; die neue Version legt jetzt die Temporärdateien mit dem Namen des Last.fm-Benutzers an, sodass für verschiedene Accounts jeweils die passend vagalume-"Last.fm-User" Datei abgelegt wird. Jetzt wieder im /tmp-Ordner, da stören sie nicht weiter (und da es jetzt ja verschiedene Namen sind, sich auch nicht gegenseitig 😉 )

Damit kann gleich für alle Instanzen das selbe Downloadverzeichnis angegeben werden.

Allerdings hab' ich noch nicht raus, wie ich Vagalume dazu bewege, die existierende Datei eines bereits laufenden Players von der nächsten nicht überschreiben zu lassen. Man müsste wohl einen check einbauen, ob die vagalume-"Last.fm-User"-Datei bereits existiert; vielleicht finde ich das noch raus, aber meine bisherigen Versuche endeten mit "segmentation-fault"... 😬
Vielleicht hat jemand eine Idee dazu? Ich denke, es wird wohl in src/audio.c, Zeilen 301 ff. (gepatchte Version, is' ja logo...) geregelt; aber mir fällt gerade nichts Gescheites ein...

Wenn allerdings die einzelnen Instanzen zunächst nur geöffnet, aber noch nicht gestartet werden, sondern erst, wenn die jeweiligen Account-Daten eingegeben sind, wird keine Datei überschrieben, sonder gleich die jeweils "richtige" angelegt.

so long
hank

Hi!

Die Lösung war relativ einfach - in src/audio.c Zeile 306 auskommentieren...

//       controller_del_dump (dumppath); // delete the previous dump file if any

Die existierende Datei wird dann nicht wie bisher schon beim Starten überschrieben; trotzdem wird eine eventuell vorhandene alte beim Songwechsel/Neustart ersetzt (ggf. bleibt bei Beenden des Players/Absturz (passiert natürlich nicht 😉 ) eine Dateileiche zurück, aber ein neu gespielter Titel auf dem Account ersetzt sie wieder, es sammelt sich also kein Müll an)

Überarbeiteter multiple-usage-patch und 32-bit-hardy-test.deb im Anhang

so long
hank

uvok89 schrieb:

Hallo,

ich hab jetzt mal versucht, das gepatchte Vagalume unter Gentoo zum laufen zu bringen... ohne Erfolg.

Welche gnome, gstreamer & glib Versionen? (vor allem gstreamer & die relevanten plugins müssen mit gnome glib 2.18 support gebaut sein, damit gnomevfs & gio funktionieren.)

Ich verwende selbst kein gentoo, aber wahrscheinlich muss erst das ebuild angepasst werden. Die gepatchte version hat einige mehr dependencies zum build als die original vagalume-0.7 von Berto. Wie die nötigen pakete in debian / ubuntu heissen findest du hier. Die gentoo Pendants lassen sich sicherlich recht einfach identifizieren. Es könnte sein dass in gentoo gio bereits in glib gemerged ist und es kein extra gio paket gibt, aber dazu bitte bei einen gentoo user infos einholen.

In Intrepid sind die relevanten build settings:

      host-triplet:	i486-pc-linux-gnu
      shell:		/bin/bash
      compiler:		gcc version 4.3.2  
      compiler flags:		-Wall -g -O2
      linker:		/usr/bin/ld (gnu? yes)
      libtool:		(GNU libtool) 2.2.4 Debian
      automake:		automake (GNU automake) 1.10.1
      autoconf:		autoconf (GNU Autoconf) 2.61

genaueres zur ubuntu gcc version:

1
2
3

gcc version 4.3.2 (Ubuntu 4.3.2-1ubuntu10)
Thread model: posix
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.3.2-1ubuntu10' --with-bugurl=file:///usr/share/doc/gcc-4.3/README.Bugs --enable-languages=c,c++,fortran,objc,obj-c++ --prefix=/usr --enable-shared --with-system-zlib --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --enable-nls --with-gxx-include-dir=/usr/include/c++/4.3 --program-suffix=-4.3 --enable-clocale=gnu --enable-libstdcxx-debug --enable-objc-gc --enable-mpfr --enable-targets=all --enable-checking=release --build=i486-linux-gnu --host=i486-linux-gnu --target=i486-linux-gnu

Heinrich Schwietering schrieb:

Überarbeiteter multiple-usage-patch und 32-bit-hardy-test.deb im Anhang

Funktioniert gut! Jetzt gibts eigentlich keinen grund mehr den patch nicht im default build zu haben ☺ Thx!

Gnome: 2.22.3 (enthält keine Dateien, wahrscheinlich nur zur Realisierung der Abhängigkeiten) gnome-common: 2.20.0 (enthält zwei *.m4-Dateien) gnome-desktop: 2.22.3 (enthält libgnome-desktop*.so) libgnome: 2.22.0 (header-Dateien) gstreamer: 0.10.20 glib: 2.18.2 glibc: 2.6.1 gst-plugins-gnomevfs: 0.10.20 Gstreamer-Plugins: 0.10.* (versch. Versionen)

GCC:

Ziel: i686-pc-linux-gnu
Konfiguriert mit: /var/tmp/portage/sys-devel/gcc-4.3.2/work/gcc-4.3.2/configure --prefix=/usr --bindir=/usr/i686-pc-linux-gnu/gcc-bin/4.3.2 --includedir=/usr/lib/gcc/i686-pc-linux-gnu/4.3.2/include --datadir=/usr/share/gcc-data/i686-pc-linux-gnu/4.3.2 --mandir=/usr/share/gcc-data/i686-pc-linux-gnu/4.3.2/man --infodir=/usr/share/gcc-data/i686-pc-linux-gnu/4.3.2/info --with-gxx-include-dir=/usr/lib/gcc/i686-pc-linux-gnu/4.3.2/include/g++-v4 --host=i686-pc-linux-gnu --build=i686-pc-linux-gnu --disable-altivec --enable-nls --without-included-gettext --with-system-zlib --disable-checking --disable-werror --enable-secureplt --disable-multilib --enable-libmudflap --disable-libssp --enable-cld --disable-libgcj --with-arch=i686 --enable-languages=c,c++,objc,treelang,fortran --enable-shared --enable-threads=posix --enable-__cxa_atexit --enable-clocale=gnu --with-bugurl=http://bugs.gentoo.org/ --with-pkgversion='Gentoo 4.3.2 p1.0'
Thread-Modell: posix
gcc-Version 4.3.2 (Gentoo 4.3.2 p1.0) 

das komplette log auch noch anhängen bitte

A complete build log is located at '/var/tmp/portage/media-sound/vagalume-0.7/temp/build.log'.

mtron schrieb:

das komplette log auch noch anhängen bitte

A complete build log is located at '/var/tmp/portage/media-sound/vagalume-0.7/temp/build.log'.

Jetzt gab es zwar keinen Violation Erro mehr, dafür kann der Befehl vagalume nicht mehr gefunden werden.

Build-Log: (Ich hab den Bereich, in dem der Fehler wohl liegt, markiert; es sollte ungefähr in der Mitte der Seite sein)

http://pastebin.com/f24d7a90e

Ich kann nur schonmal soviel sagen: Es hängt wohl mit libtool (im Vagalume-Source-Verzeichnis) zusammen, Irgendwo wurde da ein 'X' in den Compileraufruf (?) reingeschmuggelt ...

EDIT:

Wenn ich das Paket manuell kompiliere (Source entpacken, Patch anwenden, configure, make), erscheint der Fehler auch.

ABER: Wenn ich make distclean und ./autogen.sh ausführe, geht es! Ich vermute, dass ./autogen.sh die configure.ac löscht, Und die wird vom Patch bearbeitet.

Also scheint das Patchen der configure.ac den Fehler zu verursachen...

Ich weiß noch nicht, ob Vagalume dann auch läuft, wenn die configure.ac nicht gepatcht ist, das wird sich gleich zeigen...