1.51.78 deb packages from bearoso's ppa are weird

[GoogleCodeExporter]

 Packages are ~3 MB instead of ~0.9 MB in size:)... I prepared fixed
version in my ppa, so if someone want to look:

https://launchpad.net/~smoki00790/+archive/ppa/+sourcepub/917948/+listing-archiv
e-extra

  I reworked rules file, added man page, etc. It is also without any
lintian warnings/errors.

Original issue reported on code.google.com by smoki00...@gmail.com on 3 Jan 2010 at 5:16

[GoogleCodeExporter]

I did have it stripping the binary at one time. I can't remember why I removed 
it. 

I'm not actually using Debian or Ubuntu anywhere, so I had to "cross-build" the
packages. I know the entire package structure is out-of-date and fails the 
lintian
checks and doesn't use separate builds for different OS versions.

Original comment by bear...@gmail.com on 3 Jan 2010 at 9:36

• Changed state: Accepted

[GoogleCodeExporter]

 OK i was thinking that you use debs just because debian lab is in source package.
Anyway it is better for future releases to not provide that lab in the original
source package, so packagers or you can make debs easely and control it 
externaly
without any problems with warnings/errors.

 About glade patch (in my debian/patches) for removing 'orientation' property... i
see nautilus devs are removing it also. Seems to be some glade vs gtkbulder 
problem?
Also another gtk warnings are because GtkComboBox quirks in glade:

(snes9x-gtk:6291): Gtk-CRITICAL **: gtk_tree_row_reference_new: assertion
`GTK_IS_TREE_MODEL (model)' failed

(snes9x-gtk:6291): Gtk-CRITICAL **: gtk_cell_view_set_displayed_row: assertion
`GTK_IS_TREE_MODEL (cell_view->priv->model)' failed

 To avoid these warnings some suggests to not activate default items via glade file
and instead hard code it with: 

 gtk_combo_box_set_active(GTK_COMBO_BOX(cmb1), 0); 

 And must say that i'm heard of your's snes9x gui just 4 days ago, so huge thanks for
making it - you doing it very well:).

Original comment by smoki00...@gmail.com on 3 Jan 2010 at 12:02

[GoogleCodeExporter]

Ok, thanks for the info. 

I explicitly removed the orientation properties from the merge branch glade 
file, but
they must have sneaked back in there. And I do set the active items outside of 
glade,
so I'll remove the default item stuff, too.

Original comment by bear...@gmail.com on 3 Jan 2010 at 10:08

[GoogleCodeExporter]

The 1.52.79 packages are basically your scripts with the patch removed. Reopen 
this
bug if there's any problems. 

Original comment by bear...@gmail.com on 12 Jan 2010 at 10:01

• Changed state: Fixed

[GoogleCodeExporter]

Yes there is:). 1.52.79 deb packages misses pulseaudio! Just add 'libpulse-dev' 
to
Build-Depends and they will be proper. 

 I'm also lzmaed packages in my ppa, so they are now cca 30% smaller... I know i'm
fancy:).

Original comment by smoki00...@gmail.com on 12 Jan 2010 at 10:14

[GoogleCodeExporter]

Actually, the PulseAudio thing was intentional. Many users seem to go to
extraordinary lengths to remove it from their systems, and if Snes9x links 
against it
they have to have it, so I decided not to enable it by default.

That said, do you really think it would be better to enable it?

Original comment by bear...@gmail.com on 12 Jan 2010 at 10:40

[GoogleCodeExporter]

What the hell. Changed anyway.

Original comment by bear...@gmail.com on 12 Jan 2010 at 11:10

[GoogleCodeExporter]

 Yep the hell:) Of course it should be enabled so ubunteros can test it, also who
remove pulseaudio from their Ubuntu systems,  they can also recompile it w/o 
it:).
One more reason - pulseaudio is installed by default in Ubuntu, but for example
portaudio is not.

Original comment by smoki00...@gmail.com on 12 Jan 2010 at 11:18