Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 424207 - app-emulation/playonlinux-4.1.8 version bump request
Summary: app-emulation/playonlinux-4.1.8 version bump request
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal enhancement with 4 votes (vote)
Assignee: Michael Mol
URL:
Whiteboard:
Keywords:
: 434214 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-06-30 06:57 UTC by Pietro Acinapura
Modified: 2013-02-14 07:11 UTC (History)
8 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
app-emulation/playonlinux-4.1.2 - bump version (playonlinux-4.1.2.ebuild,3.09 KB, text/plain)
2012-06-30 06:57 UTC, Pietro Acinapura
Details
debian/patches/disable_update_alert.diff (disable_update_alert.diff,589 bytes, patch)
2012-06-30 09:03 UTC, poncho
Details | Diff
Bump version to playonlinux-4.1.3 (playonlinux-4.1.3.ebuild,2.95 KB, text/plain)
2012-07-15 19:57 UTC, Michael Mol
Details
Conversation with upstream regarding DEBIAN_PACKAGE flag behavior (pol_DEBIAN_PACKAGE.log,3.58 KB, text/plain)
2012-07-21 13:26 UTC, Michael Mol
Details
playonlinux-4.1.8.ebuild (playonlinux-4.1.8.tar.xz,3.08 KB, application/octet-stream)
2012-10-12 20:08 UTC, poncho
Details
playonlinux-4.1.8.ebuild Diff (version_bump.patch,3.47 KB, patch)
2012-10-12 20:10 UTC, poncho
Details | Diff
emerge --info (emerge_info,6.44 KB, text/plain)
2012-12-20 20:26 UTC, Matt
Details
playonlinux-4.1.9.ebuild (playonlinux-4.1.9.ebuild,3.18 KB, text/plain)
2013-02-13 21:30 UTC, James
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Pietro Acinapura 2012-06-30 06:57:41 UTC
Created attachment 316739 [details]
app-emulation/playonlinux-4.1.2 - bump version

app-emulation/playonlinux-4.1.2 - bump version
Comment 1 poncho 2012-06-30 09:03:48 UTC
Created attachment 316743 [details, diff]
debian/patches/disable_update_alert.diff

Perhaps you want to add debian's disable_update_alert.diff to disable the annoying "An updated version is available" popup window.

see http://anonscm.debian.org/viewvc/pkg-games/packages/trunk/playonlinux/debian/patches/disable_update_alert.diff?view=markup
Comment 2 Michael Mol 2012-07-15 19:57:55 UTC
Created attachment 318250 [details]
Bump version to playonlinux-4.1.3

This ebuild bumps us up to playonlinux 4.1.3. It also reuses the IPv6 patch we presently apply to 4.1.1. Tested on amd64.

I'm looking into the 'DEBIAN_PACKAGE' switch.
Comment 3 Michael Mol 2012-07-15 21:01:17 UTC
poncho, could you file that as a new bug request, with priority 'enhancement'?
Comment 4 Markos Chandras (RETIRED) gentoo-dev 2012-07-21 11:25:40 UTC
I presume we can handle both in this bug
Comment 5 Michael Mol 2012-07-21 13:25:04 UTC
I spent some time with upstream on the issue, and it's not entirely clear what the best way to handle it is.

I was warned that setting the 'DEBIAN_PACKAGE' variable that POL supports does more than just turn off the update notifications. Looked into it, and it:

1) Disables update notifications
2) Disables automatic bug reports
3) Disables automatic usage statistic collection
4) Changes the directory POL looks in for locale data (POL, by default, ships with a copy of locale data in its own directory structure, but setting DEBIAN_PACKAGE switches code to look at a hardcoded path. It happens that this path exists on Gentoo as well.)
5) Changes the behavior of some code which sets terminal titles.
6) Changes how POL guarantees the presence of MS Core Fonts.

Re (1). I'm not certain I really want to disable update notifications. I rather like the idea that I could instead direct the user via hyperlink to a particular bug number on b.g.o which is updated as each new version comes along. This has the benefit of avoiding duplicate bug reports, as well as ensuring the maintainer rapidly aware of the availability of a new version upstream.

If getting decent ebuild of the new version would prove to be difficult, a new revision could be added which disables the notification. Similarly, if it's desired to mark versions of app-emulation/playonlinux stable, the update notification could be removed there, too. The DEBIAN_PACKAGE flag doesn't accomplish any of this.

Re (2). Upstream is partial to disabling automatic bug reports for packaged versions of POL. I understand and agree with them on that. The DEBIAN_PACKAGE flag accomplishes this, but has additional side-effects, and so is less desireable.

Re (3). Statistic collection and submission requires the user's permission at first-run anyway, so it's not necessary to disable it from a privacy standpoint. Further, I think users' ability to inform upstream of their use case in a structured and controlled way is a benefit to those users, so I don't like DEBIAN_PACKAGE disabling that. I didn't get any argument from upstream on this. (I'll attach the relevant pieces of the chat log after I finish this comment).

Re (4). While I understand what locales do and are generally for, I don't fully understand the effect of the locale path change in this circumstance. For example, the DEBIAN_PACKAGE flag instructs the program to look under /usr/share/locale (as opposed to the program's install directory tree) for locale data. I don't know if Debian's maintainer is moving app-specific files into there or not. I really need to study up on the mechanics of locales and of potential distribution-imposed path variations before I make a change here. (I'm also in the middle of an employment shift, so while I have most of the documentation I need, the studying itself is on a back burner at this moment.)

Re (5). I don't know why DEBIAN_PACKAGE has influence on how terminal titles are adjusted. If the other issues are resolved and we may use the DEBIAN_PACKAGE flag, I will probably simply test whether the net result is unsatisfactory and go from there.

Re (6). Obviously, a dependency on media-fonts/corefonts is in order, and POL shouldn't need to check for the fonts' presence. I haven't dug deeply into the depth of POL's font install code, though, so I don't know how difficult it would be to come up with a clean patch.

Long and short of it, the DEBIAN_PACKAGE flag impacts a lot of code, and the full extent is not clear to me.

I hope to convince upstream that item-by-item behavior flags are a better way to implement their in-tree customization options than what they're doing; I've got a simple, should-be-transparent implementation for it in mind, too. If I can convince them of that approach, I'll write the relevant code and submit patches; they're very grateful for patches.

I'm also partial to keeping Gentoo's version as close to upstream as reasonably possible, with few (if any) invasive changes, to ease future maintenance. If my patches work for them, that's awesome, and each new release of upstream should result in a reduction of the number of patches Gentoo applies. If upstream doesn't like my patches, I'll go with whatever we can mutually come up with so that Gentoo users have good experiences, while the maintainer of the Gentoo package has minimal maintenance resulting from package divergence.
Comment 6 Michael Mol 2012-07-21 13:26:07 UTC
Created attachment 318818 [details]
Conversation with upstream regarding DEBIAN_PACKAGE flag behavior
Comment 7 tman 2012-08-23 05:50:17 UTC
hi,

in the meanwhile there is version 4.1.7. i could be nice if a dev can update now the ebuild to this latest version.

thanks
Comment 8 nybblenybble 2012-09-03 22:19:21 UTC
Play on Linux is now up to version 4.1.7, bump request/update on progress please?
Comment 9 Michael Mol 2012-09-04 01:29:34 UTC
I'll put together and test an ebuild this weekend. Any particular application you'd like me to test? Otherwise, I just use Diablo III.
Comment 10 Michael Mol 2012-09-09 21:13:57 UTC
Apologies, I'm not going to get to it this weekend. Fingers crossed for next weekend.
Comment 11 tman 2012-09-24 05:17:58 UTC
 version 4.1.8 released.
Comment 12 poncho 2012-10-12 20:08:38 UTC
Created attachment 326400 [details]
playonlinux-4.1.8.ebuild

This is the ebuild I am currently using.
Comment 13 poncho 2012-10-12 20:10:40 UTC
Created attachment 326402 [details, diff]
playonlinux-4.1.8.ebuild Diff

For convenience the diff against the ebuild in the main tree.
Comment 14 Gary HUnt 2012-11-24 02:07:57 UTC
Can we get this ebuild in the tree? I was going to post an ebuild for 4.1.8 as well before I saw poncho's.
Comment 15 Markos Chandras (RETIRED) gentoo-dev 2012-11-25 12:35:10 UTC
(In reply to comment #14)
> Can we get this ebuild in the tree? I was going to post an ebuild for 4.1.8
> as well before I saw poncho's.

We havent heard back from Michael Mol yet...
Comment 16 Michael Mol 2012-11-26 01:46:33 UTC
Please, go on without me; I've just purchased a house and moved, and I don't see immediate demands on my time getting any lighter for the foreseeable future.
Comment 17 Markos Chandras (RETIRED) gentoo-dev 2012-11-26 09:36:18 UTC
(In reply to comment #16)
> Please, go on without me; I've just purchased a house and moved, and I don't
> see immediate demands on my time getting any lighter for the foreseeable
> future.

So I guess we need to find a new maintainer for this. Volunteers? :)
Comment 18 Michael Weber (RETIRED) gentoo-dev 2012-11-26 13:31:04 UTC
I can volunteer, but I'm new to the package.
Do we really need netcat? Would netcat6 be ok, too? Both are in conflict.
Comment 19 Michael Mol 2012-11-26 13:41:40 UTC
As long as netcat6 handles IPv4 just fine (my Gentoo boxes are down right now, so I can't check), it shouldn't be a problem.

You'd want to set the dependency as "netcat or netcat6", though.
Comment 20 Markos Chandras (RETIRED) gentoo-dev 2012-12-02 11:29:46 UTC
(In reply to comment #18)
> I can volunteer, but I'm new to the package.
> Do we really need netcat? Would netcat6 be ok, too? Both are in conflict.

So I guess this package can be dropped from proxy-maint now?
Comment 21 Michael Behr 2012-12-20 17:10:01 UTC
changed netcat to netcat6 in your ebuild (thanks for that :-) @poncho) no problems so far.

don't know if needed:
emerge --info 

Portage 2.2.0_alpha149 (default/linux/amd64/10.0/desktop, gcc-4.5.4, glibc-2.15-r3, 3.4.9-gentoo x86_64)
=================================================================
System uname: Linux-3.4.9-gentoo-x86_64-Intel-R-_Core-TM-2_Quad_CPU_Q9550_@_2.83GHz-with-gentoo-2.1
Timestamp of tree: Wed, 19 Dec 2012 08:45:01 +0000
ld GNU ld (GNU Binutils) 2.22
ccache version 3.1.8 [enabled]
app-shells/bash:          4.2_p37
dev-java/java-config:     2.1.11-r3
dev-lang/python:          2.7.3-r2, 3.2.3
dev-util/ccache:          3.1.8
dev-util/cmake:           2.8.9
dev-util/pkgconfig:       0.27.1
sys-apps/baselayout:      2.1-r1
sys-apps/openrc:          0.11.8
sys-apps/sandbox:         2.5
sys-devel/autoconf:       2.13, 2.68
sys-devel/automake:       1.9.6-r3, 1.11.6
sys-devel/binutils:       2.22-r1
sys-devel/gcc:            4.5.4
sys-devel/gcc-config:     1.7.3
sys-devel/libtool:        2.4-r1
sys-devel/make:           3.82-r4
sys-kernel/linux-headers: 3.6 (virtual/os-headers)
sys-libs/glibc:           2.15-r3
Repositories: gentoo local_overlay
Installed sets: @kdeartwork, @system
ACCEPT_KEYWORDS="amd64"
ACCEPT_LICENSE="* -@EULA"
CBUILD="x86_64-pc-linux-gnu"
CFLAGS="-O2 -pipe -march=native -fomit-frame-pointer"
CHOST="x86_64-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/share/config /usr/share/gnupg/qualified.txt /usr/share/openvpn/easy-rsa /usr/share/themes/oxygen-gtk/gtk-2.0 /var/lib/hsqldb"
CONFIG_PROTECT_MASK="/etc/ca-certificates.conf /etc/dconf /etc/env.d /etc/env.d/java/ /etc/fonts/fonts.conf /etc/gconf /etc/gentoo-release /etc/php/apache2-php5.3/ext-active/ /etc/php/apache2-php5.4/ext-active/ /etc/php/cgi-php5.3/ext-active/ /etc/php/cgi-php5.4/ext-active/ /etc/php/cli-php5.3/ext-active/ /etc/php/cli-php5.4/ext-active/ /etc/revdep-rebuild /etc/sandbox.d /etc/terminfo /etc/texmf/language.dat.d /etc/texmf/language.def.d /etc/texmf/updmap.d /etc/texmf/web2c"
CXXFLAGS="-O2 -pipe -march=native -fomit-frame-pointer"
DISTDIR="/usr/portage/distfiles"
FCFLAGS="-O2 -pipe"
FEATURES="assume-digests binpkg-logs candy ccache config-protect-if-modified distlocks ebuild-locks fail-clean fixlafiles merge-sync metadata-transfer news parallel-fetch preserve-libs protect-owned sandbox sfperms strict unknown-features-warn unmerge-logs unmerge-orphans userfetch"
FFLAGS="-O2 -pipe"
GENTOO_MIRRORS="http://ftp.uni-erlangen.de/pub/mirrors/gentoo http://mirrors.sec.informatik.tu-darmstadt.de/gentoo/ http://gentoo.mneisen.org/ http://de-mirror.org/distro/gentoo/ http://ftp.spline.inf.fu-berlin.de/mirrors/gentoo/ "
LANG="de_DE.UTF-8"
LDFLAGS="-Wl,-O1 -Wl,--as-needed"
MAKEOPTS="-j5"
PKGDIR="/usr/portage/packages"
PORTAGE_CONFIGROOT="/"
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --compress --force --whole-file --delete --stats --human-readable --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/portage/local"
SYNC="rsync://rsync.europe.gentoo.org/gentoo-portage"
USE="X a52 aac acl acpi alsa amd64 apache2 avahi bash-completion berkdb bluetooth branding bzip2 cdda cdr cli consolekit cracklib crypt css cups curl cxx dbus divx dri dts dv dvd dvdr dvdread emboss encode exif fam ffmpeg firefox flac fortran gdbm gif gpm gtk iconv imagemagick java java6 jpeg jpeg2k kde lcms ldap libnotify lm_sensors mad mmx mng modules mp3 mp4 mpeg mplayer mtp mudflap multilib mysql ncurses nls nptl nsplugin odbc ogg opengl openmp pam pango pcre pdf php png policykit ppds pppd python qt3support qt4 quicktime readline realmedia samba sdl semantic-desktop session spell spice sqlite sqlite3 sse sse2 ssl startup-notification subtitles svg theora threads tiff tk truetype udev udisks unicode upower usb v4l v4l2 vaapi vhosts vim-syntax vorbis wavpack wmf wmp wxwidgets x264 xcb xcomposite xinerama xml xv xvid zeroconf zlib" ALSA_CARDS="ali5451 als4000 atiixp atiixp-modem bt87x ca0106 cmipci emu10k1x ens1370 ens1371 es1938 es1968 fm801 hda-intel intel8x0 intel8x0m maestro3 trident usb-audio via82xx via82xx-modem ymfpci" ALSA_PCM_PLUGINS="adpcm alaw asym copy dmix dshare dsnoop empty extplug file hooks iec958 ioplug ladspa lfloat linear meter mmap_emul mulaw multi null plug rate route share shm softvol" APACHE2_MODULES="dav dav_fs dav_lock authz_host dir mime alias auth_basic autoindex authn_file authz_user setenvif expires rewrite vhost_alias" CALLIGRA_FEATURES="kexi words flow plan sheets stage tables krita karbon braindump" CAMERAS="ptp2" COLLECTD_PLUGINS="df interface irq load memory rrdtool swap syslog" ELIBC="glibc" GPSD_PROTOCOLS="ashtech aivdm earthmate evermore fv18 garmin garmintxt gpsclock itrax mtk3301 nmea ntrip navcom oceanserver oldstyle oncore rtcm104v2 rtcm104v3 sirf superstar2 timing tsip tripmate tnt ubx" INPUT_DEVICES="evdev" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LIBREOFFICE_EXTENSIONS="presenter-console presenter-minimizer" LINGUAS="de" MISDN_CARDS="avmfritz" PHP_TARGETS="php5-3" PYTHON_SINGLE_TARGET="python2_7" PYTHON_TARGETS="python2_7 python3_2" QEMU_SOFTMMU_TARGETS="i386 x86_64" QEMU_USER_TARGETS="i386 x86_64" RUBY_TARGETS="ruby18 ruby19" USERLAND="GNU" VIDEO_CARDS="fglrx" XTABLES_ADDONS="quota2 psd pknock lscan length2 ipv4options ipset ipp2p iface geoip fuzzy condition tee tarpit sysrq steal rawnat logmark ipmark dhcpmac delude chaos account"
Unset:  CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LC_ALL, PORTAGE_BUNZIP2_COMMAND, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS, USE_PYTHON
Comment 22 Matt 2012-12-20 20:25:21 UTC
Just poking my head in to say that I've successfully used the proposed ebuild for 4.1.8 (WITH the netcat6 change) and everything is fine here.  I'm attaching my --info for reference.

THANK YOU ALL!


M.
Comment 23 Matt 2012-12-20 20:26:05 UTC
Created attachment 332832 [details]
emerge --info

emerge --info for x86 machine.
Comment 24 Michael Weber (RETIRED) gentoo-dev 2012-12-20 23:01:10 UTC
Well, I FEATURE=test build this on an blank ~amd64 system, and proceed.

@Michael Behr and Matt: Can I enthuse you for proxy-maintainers project and being assigned on new bugs? I'd review your version bump/bugfix suggestions and commit these to tree.
Comment 25 Denis M. (Phr33d0m) 2013-02-12 01:14:55 UTC
*** Bug 434214 has been marked as a duplicate of this bug. ***
Comment 26 James 2013-02-13 21:30:45 UTC
Created attachment 338804 [details]
playonlinux-4.1.9.ebuild

An ever so slightly modified ebuild (as attached earlier) for playonlinux version 4.1.9.
Comment 27 tman 2013-02-13 21:40:30 UTC
or use poly-c overlay there is an ebuild which  is mostly up to date
Comment 28 Michael Weber (RETIRED) gentoo-dev 2013-02-14 07:11:24 UTC
happy testing ...


+*playonlinux-4.1.9 (14 Feb 2013)
+
+  14 Feb 2013; Michael Weber <xmw@gentoo.org> +playonlinux-4.1.9.ebuild:
+  Version bump (bug 424207)
+