Re: [OE-core] [PATCH] pulseaudio: Update to 17.0

2024-01-24 Thread Fabio Estevam
Hi Ross,

On Wed, Jan 24, 2024 at 9:48 AM Ross Burton  wrote:
>
> On 23 Jan 2024, at 14:08, Fabio Estevam via lists.openembedded.org 
>  wrote:
> > +FILES:${PN}-server = "${bindir}/pulseaudio ${bindir}/start-* ${datadir} 
> > ${sysconfdir} ${bindir}/pactl */udev/rules.d/*.rules 
> > */*/udev/rules.d/*.rules ${systemd_user_unitdir}/*”
>
> It’s best not to add entire trees like ${datadir} to files, as there may be 
> unexpected consequences.

Thanks for the feedback.

I will send v2 that only adds the specific file instead:

FILES:${PN}-server += "${datadir}/dbus-1/system.d/pulseaudio-system.conf"

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#194270): 
https://lists.openembedded.org/g/openembedded-core/message/194270
Mute This Topic: https://lists.openembedded.org/mt/103909681/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [PATCH] pulseaudio: Update to 17.0

2024-01-24 Thread Ross Burton
On 23 Jan 2024, at 14:08, Fabio Estevam via lists.openembedded.org 
 wrote:
> +FILES:${PN}-server = "${bindir}/pulseaudio ${bindir}/start-* ${datadir} 
> ${sysconfdir} ${bindir}/pactl */udev/rules.d/*.rules */*/udev/rules.d/*.rules 
> ${systemd_user_unitdir}/*”

It’s best not to add entire trees like ${datadir} to files, as there may be 
unexpected consequences.

If you run buildhistory-diff to verify the packaging changes, then you’ll see 
this did not do what you wanted:

packages/cortexa57-poky-linux/pulseaudio/pulseaudio-doc: PKGSIZE changed from 
70855 to 0 (-100%)
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-af: FILELIST: 
removed all items "/usr/share/locale/af/LC_MESSAGES/pulseaudio.mo"
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-af: PKGSIZE changed 
from 10401 to 0 (-100%)
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-as: FILELIST: 
removed all items "/usr/share/locale/as/LC_MESSAGES/pulseaudio.mo"
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-as: PKGSIZE changed 
from 39885 to 0 (-100%)
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-be: FILELIST: 
removed all items "/usr/share/locale/be/LC_MESSAGES/pulseaudio.mo"
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-be: PKGSIZE changed 
from 60632 to 0 (-100%)
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-bg: FILELIST: 
removed all items "/usr/share/locale/bg/LC_MESSAGES/pulseaudio.mo"
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-bg: PKGSIZE changed 
from 1937 to 0 (-100%)
[ skip tens more locale packages ]
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-zh-tw: FILELIST: 
removed all items "/usr/share/locale/zh_TW/LC_MESSAGES/pulseaudio.mo"
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-locale-zh-tw: PKGSIZE 
changed from 57726 to 0 (-100%)
packages/cortexa57-poky-linux/pulseaudio/pulseaudio-server: FILELIST: directory 
renamed /etc/dbus-1/system.d -> /usr/share/dbus-1/system.d, added 
"/usr/share/locale/sv/LC_MESSAGES/pulseaudio.mo 
/usr/share/bash-completion/completions/pulseaudio 
/usr/share/locale/el/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/uk/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/be/LC_MESSAGES/pulseaudio.mo 
/usr/share/man/man1/start-pulseaudio-x11.1 
/usr/share/locale/nl/LC_MESSAGES/pulseaudio.mo /usr/share/man/man1/pamon.1 
/usr/share/bash-completion/completions/pactl 
/usr/share/vala/vapi/libpulse-simple.deps 
/usr/share/locale/he/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/ja/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/nn/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/pl/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/pt_BR/LC_MESSAGES/pulseaudio.mo 
/usr/share/bash-completion/completions/pasuspender 
/usr/share/locale/kk/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/eu/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/bg/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/kn/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/hu/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/oc/LC_MESSAGES/pulseaudio.mo 
/usr/share/man/man1/pax11publish.1 /usr/share/man/man5/pulse-daemon.conf.5 
/usr/share/locale/sr/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/pt/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/id/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/gl/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/hi/LC_MESSAGES/pulseaudio.mo /usr/share/man/man5/default.pa.5 
/usr/share/bash-completion/completions/paplay 
/usr/share/locale/hr/LC_MESSAGES/pulseaudio.mo 
/usr/share/vala/vapi/libpulse.vapi 
/usr/share/locale/pa/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/zh_CN/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/de/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/as/LC_MESSAGES/pulseaudio.mo /usr/share/man/man1/padsp.1 
/usr/share/locale/es/LC_MESSAGES/pulseaudio.mo /usr/share/man/man1/parec.1 
/usr/share/locale/af/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/ml/LC_MESSAGES/pulseaudio.mo /usr/share/man/man1/pacat.1 
/usr/share/locale/ru/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/it/LC_MESSAGES/pulseaudio.mo 
/usr/share/bash-completion/completions/parecord 
/usr/share/man/man5/pulse-cli-syntax.5 
/usr/share/locale/lt/LC_MESSAGES/pulseaudio.mo /usr/share/man/man1/parecord.1 
/usr/share/bash-completion/completions/pacmd 
/usr/share/locale/da/LC_MESSAGES/pulseaudio.mo /usr/share/man/man1/pactl.1 
/usr/share/locale/tr/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/de_CH/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/si/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/zh_TW/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/gu/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/fr/LC_MESSAGES/pulseaudio.mo /usr/share/man/man1/pacmd.1 
/usr/share/man/man1/paplay.1 /usr/share/locale/sk/LC_MESSAGES/pulseaudio.mo 
/usr/share/bash-completion/completions/padsp 
/usr/share/locale/ca/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/eo/LC_MESSAGES/pulseaudio.mo 
/usr/share/locale/mr/LC_MESSAGES/pulseaudio.mo 
/usr/share/vala/vapi/libpulse-mainloop-glib.deps 
/usr/share/man/man1/pasuspender.1 
/usr/sh