Skip to content

Commit

Permalink
Merge remote-tracking branch 'MiczFlor/develop' into feature/mopidy-s…
Browse files Browse the repository at this point in the history
…potify-fixes
  • Loading branch information
AlvinSchiller committed Apr 9, 2024
2 parents 837fbbd + 62eed5f commit 47c23d8
Show file tree
Hide file tree
Showing 17 changed files with 23 additions and 59 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pythonpackage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
strategy:
max-parallel: 4
matrix:
python-version: ['3.7', '3.8', '3.9', '3.10', '3.11']
python-version: ['3.9', '3.10', '3.11', '3.12']

steps:
- uses: actions/checkout@v4
Expand Down
15 changes: 0 additions & 15 deletions .github/workflows/test_docker_debian.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,18 +69,3 @@ jobs:
# with:
# debian_codename: 'bullseye'
# platform: linux/arm/v6

run_buster_armv7:
name: 'buster armv7'
uses: ./.github/workflows/test_docker_debian_codename_sub.yml
with:
debian_codename: 'buster'
platform: linux/arm/v7

# # can be activate on test branches, currently failing
# run_buster_armv6:
# name: 'buster armv6'
# uses: ./.github/workflows/test_docker_debian_codename_sub.yml
# with:
# debian_codename: 'buster'
# platform: linux/arm/v6
4 changes: 2 additions & 2 deletions .github/workflows/test_docker_debian_codename_sub.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ jobs:
uses: docker/[email protected]

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3.1.0
uses: docker/setup-buildx-action@v3.2.0
with:
# network=host driver-opt needed to push to local registry
driver-opts: network=host
Expand Down Expand Up @@ -155,7 +155,7 @@ jobs:
uses: docker/[email protected]

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3.1.0
uses: docker/setup-buildx-action@v3.2.0

- name: Artifact Download Docker Image
uses: actions/download-artifact@v4
Expand Down
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ If the code change results in a test failure, we will make our best effort to co
### Guidelines
* The oldest supported Raspberry Pi OS version is currently **Buster**. Therefore all Python code should work with **Python 3.7**.
* All code has to run under the stable and legacy version of Raspberry Pi OS (please check if currently even an older version is still supported).
* For GPIO all code should work with **RPi.GPIO**. gpiozero is currently not intended to use.
### Additional Resources
Expand Down
4 changes: 2 additions & 2 deletions components/smart-home-automation/MQTT-protocol/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -144,15 +144,15 @@ components/smart-home-automation/MQTT-protocol/
## Auto-Starting the daemon at bootup

* The daemon is run by executing the script `daemon_mqtt_client.py` which will run in an endless loop.
* There's a sample service file (`phoniebox-mqtt-client.service.stretch-default.sample`) that can be used to register the daemon to be run at bootup.
* There's a sample service file (`phoniebox-mqtt-client.service-default.sample`) that can be used to register the daemon to be run at bootup.
* It is currently not integrated into the one-line-install script so please run the following commands to do it manually.

First step: copy files to destination locations:

~~~bash
# First copy the daemon script and service config file to the correct directory:
sudo cp /home/pi/RPi-Jukebox-RFID/components/smart-home-automation/MQTT-protocol/daemon_mqtt_client.py /home/pi/RPi-Jukebox-RFID/scripts/
sudo cp /home/pi/RPi-Jukebox-RFID/components/smart-home-automation/MQTT-protocol/phoniebox-mqtt-client.service.stretch-default.sample /etc/systemd/system/phoniebox-mqtt-client.service
sudo cp /home/pi/RPi-Jukebox-RFID/components/smart-home-automation/MQTT-protocol/phoniebox-mqtt-client.service-default.sample /etc/systemd/system/phoniebox-mqtt-client.service
# make sure the file exists
touch /home/pi/RPi-Jukebox-RFID/settings/Latest_RFID
~~~
Expand Down
File renamed without changes.
1 change: 0 additions & 1 deletion misc/sampleconfigs/deviceName.txt.stretch-default.sample

This file was deleted.

File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

This file was deleted.

File renamed without changes.
18 changes: 9 additions & 9 deletions ...pleconfigs/smb.conf.buster-default.sample → misc/sampleconfigs/smb.conf-default.sample
100755 → 100644
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
#
# This is the main Samba configuration file. You should read the
# smb.conf(5) manual page in order to understand the options listed
# here. Samba has a huge number of configurable options most of which
# here. Samba has a huge number of configurable options most of which
# are not shown in this example
#
# Some options that are often worth tuning have been included as
Expand All @@ -16,8 +16,8 @@
# enough to be mentioned here
#
# NOTE: Whenever you modify this file you should run the command
# "testparm" to check that you have not made any basic syntactic
# errors.
# "testparm" to check that you have not made any basic syntactic
# errors.

#======================= Global Settings =======================

Expand Down Expand Up @@ -66,7 +66,7 @@
# Server role. Defines in which mode Samba will operate. Possible
# values are "standalone server", "member server", "classic primary
# domain controller", "classic backup domain controller", "active
# directory domain controller".
# directory domain controller".
#
# Most people will want "standalone server" or "member server".
# Running as "active directory domain controller" will require first
Expand Down Expand Up @@ -101,7 +101,7 @@
#
# The following settings only takes effect if 'server role = primary
# classic domain controller', 'server role = backup domain controller'
# or 'domain logons' is set
# or 'domain logons' is set
#

# It specifies the location of the user's
Expand Down Expand Up @@ -130,13 +130,13 @@
# password; please adapt to your needs
; add user script = /usr/sbin/adduser --quiet --disabled-password --gecos "" %u

# This allows machine accounts to be created on the domain controller via the
# SAMR RPC pipe.
# This allows machine accounts to be created on the domain controller via the
# SAMR RPC pipe.
# The following assumes a "machines" group exists on the system
; add machine script = /usr/sbin/useradd -g machines -c "%u machine account" -d /var/lib/samba -s /bin/false %u

# This allows Unix groups to be created on the domain controller via the SAMR
# RPC pipe.
# RPC pipe.
; add group script = /usr/sbin/addgroup --force-badname %g

############ Misc ############
Expand Down Expand Up @@ -220,7 +220,7 @@
create mask=0777
directory mask=0777
public=no

# if the audiofiles are not in 'shared', we need this
[phoniebox_audiofile]
comment= Pi Jukebox
Expand Down
20 changes: 8 additions & 12 deletions scripts/installscripts/install-jukebox.sh
Original file line number Diff line number Diff line change
Expand Up @@ -785,7 +785,7 @@ samba_config() {
echo "Configuring Samba..."
# Samba configuration settings
# -rw-r--r-- 1 root root 9416 Apr 30 09:02 /etc/samba/smb.conf
sudo cp "${jukebox_dir}"/misc/sampleconfigs/smb.conf.buster-default.sample ${smb_conf}
sudo cp "${jukebox_dir}"/misc/sampleconfigs/smb.conf-default.sample ${smb_conf}
sudo chown root:root "${smb_conf}"
sudo chmod 644 "${smb_conf}"
# for $DIRaudioFolders using | as alternate regex delimiter because of the folder path slash
Expand All @@ -807,21 +807,21 @@ web_server_config() {
sudo chmod o+x ${HOME_DIR}
# Web server configuration settings
# -rw-r--r-- 1 root root 1040 Apr 30 09:19 /etc/lighttpd/lighttpd.conf
sudo cp "${jukebox_dir}"/misc/sampleconfigs/lighttpd.conf.buster-default.sample "${lighthttpd_conf}"
sudo cp "${jukebox_dir}"/misc/sampleconfigs/lighttpd.conf-default.sample "${lighthttpd_conf}"
sudo chown root:root "${lighthttpd_conf}"
sudo chmod 644 "${lighthttpd_conf}"
# double quotes for variable expansion
sudo sed -i "s%/home/pi%${HOME_DIR}%g" "${lighthttpd_conf}"

# Web server PHP7 fastcgi conf
# -rw-r--r-- 1 root root 398 Apr 30 09:35 /etc/lighttpd/conf-available/15-fastcgi-php.conf
sudo cp "${jukebox_dir}"/misc/sampleconfigs/15-fastcgi-php.conf.buster-default.sample ${fastcgi_php_conf}
sudo cp "${jukebox_dir}"/misc/sampleconfigs/15-fastcgi-php.conf-default.sample ${fastcgi_php_conf}
sudo chown root:root "${fastcgi_php_conf}"
sudo chmod 644 "${fastcgi_php_conf}"

# settings for php.ini to support upload
# -rw-r--r-- 1 root root 70999 Jun 14 13:50 /etc/php/7.3/cgi/php.ini
sudo cp "${jukebox_dir}"/misc/sampleconfigs/php.ini.buster-default.sample ${php_ini}
sudo cp "${jukebox_dir}"/misc/sampleconfigs/php.ini-default.sample ${php_ini}
sudo chown root:root "${php_ini}"
sudo chmod 644 "${php_ini}"

Expand Down Expand Up @@ -1034,7 +1034,7 @@ install_main() {
sudo systemctl disable phoniebox-rotary-encoder
sudo systemctl disable phoniebox-gpio-buttons.service
sudo rm "${systemd_dir}"/rfid-reader.service
sudo rm "${systemd_dir}"/startup-sound.service
sudo rm "${systemd_dir}"/phoniebox-startup-sound.service
sudo rm "${systemd_dir}"/gpio-buttons.service
sudo rm "${systemd_dir}"/idle-watchdog.service
sudo rm "${systemd_dir}"/phoniebox-rotary-encoder.service
Expand All @@ -1043,14 +1043,12 @@ install_main() {

# 2. install new ones - this is version > 1.1.8-beta
RFID_READER_SERVICE="${systemd_dir}/phoniebox-rfid-reader.service"
sudo cp "${jukebox_dir}"/misc/sampleconfigs/phoniebox-rfid-reader.service.stretch-default.sample "${RFID_READER_SERVICE}"
sudo cp "${jukebox_dir}"/misc/sampleconfigs/phoniebox-rfid-reader.service-default.sample "${RFID_READER_SERVICE}"
# Replace homedir; double quotes for variable expansion
sudo sed -i "s%/home/pi%${HOME_DIR}%g" "${RFID_READER_SERVICE}"

#startup sound now part of phoniebox-startup-scripts
#sudo cp "${jukebox_dir}"/misc/sampleconfigs/phoniebox-startup-sound.service.stretch-default.sample "${systemd_dir}"/phoniebox-startup-sound.service
STARTUP_SCRIPT_SERVICE="${systemd_dir}/phoniebox-startup-scripts.service"
sudo cp "${jukebox_dir}"/misc/sampleconfigs/phoniebox-startup-scripts.service.stretch-default.sample "${STARTUP_SCRIPT_SERVICE}"
sudo cp "${jukebox_dir}"/misc/sampleconfigs/phoniebox-startup-scripts.service-default.sample "${STARTUP_SCRIPT_SERVICE}"
# Replace homedir; double quotes for variable expansion
sudo sed -i "s%/home/pi%${HOME_DIR}%g" "${STARTUP_SCRIPT_SERVICE}"

Expand All @@ -1071,8 +1069,6 @@ install_main() {
# enable the services needed
sudo systemctl enable phoniebox-idle-watchdog
sudo systemctl enable phoniebox-rfid-reader
#startup sound is part of phoniebox-startup-scripts now
#sudo systemctl enable phoniebox-startup-sound
sudo systemctl enable phoniebox-startup-scripts
# copy mp3s for startup and shutdown sound to the right folder
cp "${jukebox_dir}"/misc/sampleconfigs/startupsound.mp3.sample "${jukebox_dir}"/shared/startupsound.mp3
Expand Down Expand Up @@ -1172,7 +1168,7 @@ wifi_settings() {
# DHCP configuration settings
local dhcpcd_conf="/etc/dhcpcd.conf"
#-rw-rw-r-- 1 root netdev 0 Apr 17 11:25 /etc/dhcpcd.conf
sudo cp "${jukebox_dir}"/misc/sampleconfigs/dhcpcd.conf.buster-default-noHotspot.sample "${dhcpcd_conf}"
sudo cp "${jukebox_dir}"/misc/sampleconfigs/dhcpcd.conf-default-noHotspot.sample "${dhcpcd_conf}"
# Change IP for router and Phoniebox
sudo sed -i 's/%WIFIinterface%/'"$WIFI_INTERFACE"'/' "${dhcpcd_conf}"
sudo sed -i 's/%WIFIip%/'"$WIFIip"'/' "${dhcpcd_conf}"
Expand Down

0 comments on commit 47c23d8

Please sign in to comment.