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

fix: adapt setup scripts to run properly on distrobox #1687

Merged
merged 7 commits into from
Nov 6, 2024
Merged

Conversation

imobachgs
Copy link
Contributor

@imobachgs imobachgs commented Oct 21, 2024

Due to a problem with ruby-augeas and Ruby 3.3, the setup script does not work anymore. In our continuous integration we decided to use the system libraries, which causes bundler to install software into /usr/lib. Of course, we do not want that behavior everywhere.

Alternatively, this PR adapts the script to work using distrobox. The setup is rather simple:

distrobox create --root --init --image tumbleweed --name agama
distrobox enter --root agama
./setup.sh

The script detects that it is running on distrobox and it workarounds the augeas problem like in our continuous integration.

Please, see #1685 for more context.

@dgdavid
Copy link
Contributor

dgdavid commented Oct 21, 2024

As said in IRC, it looks good to me. What is more, I was able to run the project in a similar way by using distrobox (thanks for the help 🙏 ). But I guess this distrobox thingy should be document as well since there could be people like me that have no idea about such a tool.

Please, see #1687 for more context.

Do you mean #1685?

@imobachgs
Copy link
Contributor Author

JFYI, I still fix a problem with NetworkManager.

@imobachgs
Copy link
Contributor Author

I have adapted the script to use the host's D-Bus system socket when running on Distrobox.

@imobachgs imobachgs marked this pull request as ready for review November 5, 2024 16:27
setup-services.sh Outdated Show resolved Hide resolved
setup-services.sh Outdated Show resolved Hide resolved
@imobachgs imobachgs merged commit 016a4d9 into master Nov 6, 2024
4 checks passed
@imobachgs imobachgs deleted the distrobox branch November 6, 2024 13:03
@imobachgs imobachgs mentioned this pull request Jan 10, 2025
imobachgs added a commit that referenced this pull request Jan 13, 2025
Update to release version 11.

* #1495
* #1564
* #1617
* #1618
* #1625
* #1626
* #1627
* #1628
* #1630
* #1631
* #1632
* #1633
* #1634
* #1635
* #1636
* #1639
* #1640
* #1641
* #1642
* #1643
* #1644
* #1645
* #1646
* #1647
* #1648
* #1649
* #1650
* #1651
* #1652
* #1654
* #1655
* #1656
* #1657
* #1660
* #1663
* #1666
* #1667
* #1668
* #1670
* #1671
* #1673
* #1674
* #1675
* #1676
* #1677
* #1681
* #1682
* #1683
* #1684
* #1687
* #1688
* #1689
* #1690
* #1691
* #1692
* #1693
* #1694
* #1695
* #1696
* #1698
* #1699
* #1702
* #1703
* #1704
* #1705
* #1707
* #1708
* #1709
* #1710
* #1711
* #1712
* #1713
* #1714
* #1715
* #1716
* #1717
* #1718
* #1720
* #1721
* #1722
* #1723
* #1727
* #1728
* #1729
* #1731
* #1732
* #1733
* #1734
* #1735
* #1736
* #1737
* #1740
* #1741
* #1743
* #1744
* #1745
* #1746
* #1751
* #1753
* #1754
* #1755
* #1757
* #1762
* #1763
* #1764
* #1765
* #1766
* #1767
* #1769
* #1771
* #1772
* #1773
* #1774
* #1777
* #1778
* #1785
* #1786
* #1787
* #1788
* #1789
* #1790
* #1791
* #1792
* #1793
* #1794
* #1795
* #1796
* #1797
* #1798
* #1799
* #1800
* #1802
* #1803
* #1804
* #1805
* #1807
* #1808
* #1809
* #1810
* #1811
* #1812
* #1814
* #1815
* #1821
* #1822
* #1823
* #1824
* #1825
* #1826
* #1827
* #1828
* #1830
* #1831
* #1832
* #1833
* #1834
* #1835
* #1836
* #1837
* #1838
* #1839
* #1840
* #1841
* #1842
* #1843
* #1844
* #1845
* #1847
* #1848
* #1849
* #1850
* #1851
* #1854
* #1855
* #1856
* #1857
* #1860
* #1861
* #1863
* #1864
* #1865
* #1866
* #1867
* #1871
* #1872
* #1873
* #1875
* #1876
* #1877
* #1878
* #1880
* #1881
* #1882
* #1883
* #1884
* #1885
* #1886
* #1888
* #1889
* #1890
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants