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

sayonara player: init at 0.9.3 #19509

Closed
wants to merge 1 commit into from
Closed

sayonara player: init at 0.9.3 #19509

wants to merge 1 commit into from

Conversation

AndersonTorres
Copy link
Member

Motivation for this change

New package: sayonara player

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • OS X
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@@ -14301,6 +14301,9 @@ in
vte = gnome3.vte;
};

sayonara = callPackage ../applications/audio/sayonara {
inherit (gst_all_1) gst-plugins-base gst-plugins-bad gst-plugins-ugly; };
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inheriting gstreamer might also make sense.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done! In fact I didn't know it was possible.

@Mic92
Copy link
Member

Mic92 commented Oct 14, 2016

I was not able to play any file, do you?
It does not register itself as a sound source in pulseaudio

@AndersonTorres
Copy link
Member Author

@Mic92 I am playing normally here. It's a screenshot I have taken just now:

https://imgur.com/a/2LVei

@Mic92
Copy link
Member

Mic92 commented Oct 16, 2016

does it work for you? @groxxda

@grahamc
Copy link
Member

grahamc commented Oct 18, 2016

I tried using it, but it didn't output any sound. I checked Devices under File, and there were no devices detected either.

@AndersonTorres
Copy link
Member Author

News?

@Mic92
Copy link
Member

Mic92 commented Oct 31, 2016

@AndersonTorres well, two people did not managed to output any sound.

@cryptix
Copy link
Contributor

cryptix commented Nov 16, 2016

Hmm.. also tried this and didn't get any playback working, devices selection empty as well.

I also noticed lots of sqlite errors like this:

Error: SQL ERROR: Store setting: Cannot insert setting remote_control_port
Error: attempt to write a readonly database Unable to fetch row
Error: INSERT INTO settings VALUES(:key, :val);

@pSub
Copy link
Member

pSub commented Jan 12, 2017

Just from looking at the error that @cryptix pasted, I guess that the program tries do an update of a sqlite database that is in nix-store and thus read-only.

sha256 = "0ykb05yv9r8568iidf7lp0dx2igfyw3d44vjsv6i3k4y0yhzdgyj";
};

buildInputs = [ pkgconfig cmake qt5.qtbase qt5.qttools libmtp taglib zlib
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pkgconfig and cmake can live in nativeBuildInputs.

'';

meta = with stdenv.lib;{
homapege = "https://sayonara-player.com/";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick: You don't need quotes around URLs.

@joachifm
Copy link
Contributor

What remains to do on this?

@AndersonTorres
Copy link
Member Author

I don't know. It worked fine here in my machine, but I can't know why it is failing on others.

@AndersonTorres
Copy link
Member Author

Now the baddies: I updated the sources, but now it isn't working anymore: it doesn't find any sound cards.

@AndersonTorres AndersonTorres changed the title sayonara player: init at 0.9.2 sayonara player: init at 0.9.3 May 13, 2017
@AndersonTorres
Copy link
Member Author

Middle news, if someone is interested: I am in contact with the Sayonara team (specifically Lucio Carreras, the main programmer). I hope to return with news!

@AndersonTorres
Copy link
Member Author

As I have no more hope working on it, I will close it without resolution of merit. Maybe another time I will consider it more closely.

@AndersonTorres AndersonTorres deleted the sayonara branch February 23, 2018 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants