Merge lp:~bennabiy/ltsp/ltsp-LinuxMint into lp:~ltsp-upstream/ltsp/ltsp-trunk

Proposed by ben-Nabiy Derush
Status: Superseded
Proposed branch: lp:~bennabiy/ltsp/ltsp-LinuxMint
Merge into: lp:~ltsp-upstream/ltsp/ltsp-trunk
Diff against target: 113 lines (+46/-8)
5 files modified
common/ltsp-common-functions (+12/-2)
server/share/ltsp/plugins/ltsp-build-client/Debian/000-basic-configuration (+12/-2)
server/share/ltsp/plugins/ltsp-build-client/Ubuntu/000-basic-configuration (+13/-3)
server/share/ltsp/plugins/ltsp-build-client/Ubuntu/010-updates-mirrors (+6/-0)
server/share/ltsp/plugins/ltsp-build-client/Ubuntu/030-artwork (+3/-1)
To merge this branch: bzr merge lp:~bennabiy/ltsp/ltsp-LinuxMint
Reviewer Review Type Date Requested Status
Vagrant Cascadian Needs Fixing
Review via email: mp+183364@code.launchpad.net

This proposal has been superseded by a proposal from 2013-09-25.

Commit message

Linux Mint support for Mint from 13 - Latest

Description of the change

This branch enables LinuxMint native support in ltsp-build-client. It enables a user of Linux Mint 13 through current (15) to build a client without getting an error. Up until now, if a user of Linux Mint wanted to build a client, they would need to ln -s /usr/share/ltsp/plugins/ltsp-build-client/Ubuntu /usr/share/ltsp/plugins/ltsp-build-client/LinuxMint and then issue the command with a --dist=(Matching Ubuntu distribution ) e.g. ltsp-build-client --dist=precise (If the machine was Linux Mint 13 (LTS) )

As of current, the building of the client is still mainly an Ubuntu client, but with the Mint repositories made available.

To test - Install Linux Mint {13,14,15}, apt-get install ltsp-standalone-server openssh-server as per UbuntuLTSP install. If you immediately build a client, it will say unknown dist (mint dist).

After applying these changes, you could build a client from a stock install of LTSP.

To post a comment you must log in.
Revision history for this message
Vagrant Cascadian (vagrantc) wrote :

Most plugins seem to be copies of ubuntu plugins, but should rather be symlinks to the appropriate plugins. Some of the plugins should also be reviewed for appropriateness for Mint- the 030-artwork plugin references ubuntu-specific packages. Good start on a proof of concept, though!

review: Needs Fixing
lp:~bennabiy/ltsp/ltsp-LinuxMint updated
2487. By ben-Nabiy Derush

Made links to Ubuntu files which differ from the Debian config, but are the same in Ubuntu config.

Revision history for this message
Vagrant Cascadian (vagrantc) wrote :

just remove 030-artwork, and review other symlinks to make sure they're valid for mint, and it looks good.

Revision history for this message
ben-Nabiy Derush (bennabiy) wrote :

> just remove 030-artwork, and review other symlinks to make sure they're valid
> for mint, and it looks good.

Done. Should be ready as of last push.

lp:~bennabiy/ltsp/ltsp-LinuxMint updated
2488. By ben-Nabiy Derush

Removed Ubuntu specific artwork reference, and linked 030-fat-client to Debian

2489. By ben-Nabiy Derush

Patched detect_vendor() to correct VENDOR and updated scripts within parent distro to detect LinuxMint and apply special settings.

2490. By ben-Nabiy Derush

Removed LinuxMint plugin folder.

2491. By ben-Nabiy Derush

Fixed vendor_detect() patch

2492. By ben-Nabiy Derush

Added missing " on line 8

2493. By ben-Nabiy Derush

Patches made according to vagrantc's suggestions.

2494. By ben-Nabiy Derush

Decomplication of 030-artwork
This line and the following will be ignored --------------

modified:
  server/share/ltsp/plugins/ltsp-build-client/Ubuntu/030-artwork

2495. By ben-Nabiy Derush

Whitespace war...

2496. By ben-Nabiy Derush

More changes... Removed whitespace tabs, etc.

2497. By ben-Nabiy Derush

Try again! More whitespace war...

2498. By ben-Nabiy Derush

INDENTured...

2499. By ben-Nabiy Derush

Almost there...

2500. By ben-Nabiy Derush

Is this the one? Updated detect_vendor() and renamed various variables...

2501. By ben-Nabiy Derush

modified:
  server/share/ltsp/plugins/ltsp-build-client/Ubuntu/000-basic-configuration

2502. By ben-Nabiy Derush

Typo

2503. By ben-Nabiy Derush

Updating with Upstream

2504. By ben-Nabiy Derush

Bug Squashing - Ubuntu distribution not being selected.

2505. By ben-Nabiy Derush

Fixing variables in 000-basic-configuration

2506. By ben-Nabiy Derush

changed case on lsb_vendor variable to reflect local, rather than global

2507. By ben-Nabiy Derush

Bug-Fix escaped ( )

2508. By ben-Nabiy Derush

Fix per alkisg: change order of tests.

2509. By ben-Nabiy Derush

Reverted 000-basic-configuration and added 001-set-mint-dist

Unmerged revisions

2509. By ben-Nabiy Derush

Reverted 000-basic-configuration and added 001-set-mint-dist

2508. By ben-Nabiy Derush

Fix per alkisg: change order of tests.

2507. By ben-Nabiy Derush

Bug-Fix escaped ( )

2506. By ben-Nabiy Derush

changed case on lsb_vendor variable to reflect local, rather than global

2505. By ben-Nabiy Derush

Fixing variables in 000-basic-configuration

2504. By ben-Nabiy Derush

Bug Squashing - Ubuntu distribution not being selected.

2503. By ben-Nabiy Derush

Updating with Upstream

2502. By ben-Nabiy Derush

Typo

2501. By ben-Nabiy Derush

modified:
  server/share/ltsp/plugins/ltsp-build-client/Ubuntu/000-basic-configuration

2500. By ben-Nabiy Derush

Is this the one? Updated detect_vendor() and renamed various variables...

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'common/ltsp-common-functions'
2--- common/ltsp-common-functions 2013-01-14 20:18:23 +0000
3+++ common/ltsp-common-functions 2013-09-25 04:36:02 +0000
4@@ -97,12 +97,22 @@
5 }
6
7 detect_vendor() {
8- if [ -e /etc/sysconfig/ltspdist ]; then
9+ CURVEND=$(lsb_release --id --short | tr " " "_")
10+ case $CURVEND in
11+ LinuxMint)
12+ if [ $(lsb_release -s -c) = "debian" ]; then
13+ VENDOR="Debian"
14+ else
15+ VENDOR="Ubuntu"
16+ fi;;
17+ *)
18+ if [ -e /etc/sysconfig/ltspdist ]; then
19 . /etc/sysconfig/ltspdist
20 echo "$VENDORDEF"
21 else
22 echo $(lsb_release --id --short | tr " " "_")
23- fi
24+ fi;;
25+ esac
26 }
27
28 # Distros may override this function to implement their own
29
30=== modified file 'server/share/ltsp/plugins/ltsp-build-client/Debian/000-basic-configuration'
31--- server/share/ltsp/plugins/ltsp-build-client/Debian/000-basic-configuration 2012-05-14 18:44:39 +0000
32+++ server/share/ltsp/plugins/ltsp-build-client/Debian/000-basic-configuration 2013-09-25 04:36:02 +0000
33@@ -2,8 +2,18 @@
34 configure)
35 ARCH=${ARCH:-"$(dpkg --print-architecture)"}
36 APT_GET_OPTS=${APT_GET_OPTS:-"-y"}
37- DIST=${DIST:-"$(lsb_release -s -c)"}
38- MIRROR=${MIRROR:-"http://cdn.debian.net/debian"}
39+
40+ # TODO: Code below is to fix LinuxMint
41+ if [ $(lsb_release -s -i) = "LinuxMint" ]; then
42+ MINTDIST=${DIST:-"$(lsb_release -s -c)"}
43+ DIST=${DIST:-"sid")
44+ MINT_MIRROR="http://packages.linuxmint.com"
45+ MINTCOMPONENTS="main upstream import"
46+ else
47+ DIST=${DIST:-"$(lsb_release -s -c)"}
48+ COMPONENTS=${COMPONENTS:-"main"}
49+ fi
50+ MIRROR=${MIRROR:-"http://cdn.debian.net/debian"}
51 COMPONENTS=${COMPONENTS:-"main"}
52 EARLY_PACKAGES=${EARLY_PACKAGES:-"ltsp-client"}
53 DEBOOTSTRAPOPTS=${DEBOOTSTRAPOPTS:-"--variant=minbase"}
54
55=== modified file 'server/share/ltsp/plugins/ltsp-build-client/Ubuntu/000-basic-configuration'
56--- server/share/ltsp/plugins/ltsp-build-client/Ubuntu/000-basic-configuration 2012-08-02 17:57:00 +0000
57+++ server/share/ltsp/plugins/ltsp-build-client/Ubuntu/000-basic-configuration 2013-09-25 04:36:02 +0000
58@@ -2,9 +2,19 @@
59 configure)
60 ARCH=${ARCH:-"$(dpkg --print-architecture)"}
61 APT_GET_OPTS=${APT_GET_OPTS:-"-y"}
62- DIST=${DIST:-"$(lsb_release -s -c)"}
63- COMPONENTS=${COMPONENTS:-"main restricted universe multiverse"}
64- EARLY_PACKAGES=${EARLY_PACKAGES:-"ltsp-client"}
65+ if [ $(lsb_release -s -i) = "LinuxMint" ]; then
66+ MINTDIST=${DIST:-"$(lsb_release -s -c)"}
67+ DIST=${DIST:-"$(grep base_codename /usr/share/mintsources/`lsb_release -c -s`/mintsources.conf | awk '{split($0,array,"=")} END{print array[2]}')}
68+ MINT_MIRROR="http://packages.linuxmint.com"
69+ MINTCOMPONENTS="main upstream import"
70+ EARLY_PACKAGES=${EARLY_PACKAGES:-"ltsp-client mint-meta-core"}
71+ LATE_PACKAGES=${LATE_PACKAGES:-"mint-meta-mate"}
72+ else
73+ DIST=${DIST:-"$(lsb_release -s -c)"}
74+ COMPONENTS=${COMPONENTS:-"main restricted universe multiverse"}
75+ EARLY_PACKAGES=${EARLY_PACKAGES:-"ltsp-client"}
76+ fi
77+
78 OVERRIDE_EXPORTS=${OVERRIDE_EXPORTS:-"true"}
79 export DEBIAN_PRIORITY=critical
80 BOOTSTRAP=${BOOTSTRAP:-"debootstrap"}
81
82=== modified file 'server/share/ltsp/plugins/ltsp-build-client/Ubuntu/010-updates-mirrors'
83--- server/share/ltsp/plugins/ltsp-build-client/Ubuntu/010-updates-mirrors 2012-08-02 17:57:00 +0000
84+++ server/share/ltsp/plugins/ltsp-build-client/Ubuntu/010-updates-mirrors 2013-09-25 04:36:02 +0000
85@@ -22,7 +22,13 @@
86 fi
87 ;;
88 after-install)
89+ if [ -n $MINTDIST ]; then
90+ add_mirrors "$MINT_MIRROR $MINTDIST $MINTCOMPONENTS"
91 add_mirrors "$UPDATES_MIRROR $DIST-updates $COMPONENTS"
92 add_mirrors "$SECURITY_MIRROR $DIST-security $COMPONENTS"
93+ else
94+ add_mirrors "$UPDATES_MIRROR $DIST-updates $COMPONENTS"
95+ add_mirrors "$SECURITY_MIRROR $DIST-security $COMPONENTS"
96+ fi
97 ;;
98 esac
99
100=== modified file 'server/share/ltsp/plugins/ltsp-build-client/Ubuntu/030-artwork'
101--- server/share/ltsp/plugins/ltsp-build-client/Ubuntu/030-artwork 2012-05-14 18:44:39 +0000
102+++ server/share/ltsp/plugins/ltsp-build-client/Ubuntu/030-artwork 2013-09-25 04:36:02 +0000
103@@ -2,7 +2,9 @@
104 finalization)
105 if [ -n "$FAT_CLIENT_DESKTOPS" ]; then
106 desktops="$FAT_CLIENT_DESKTOPS"
107- else
108+ elif [ -n "$MINTDIST" ]; then
109+ desktops=$(dpkg -l | sed -n 's/^ii.*\<\(.*mate-desktop\)\>.*/\1/p')
110+ else
111 desktops=$(dpkg -l | sed -n 's/^ii.*\<\(.*buntu-desktop\)\>.*/\1/p')
112 fi
113 # Prefer artwork for desktops other than then default ubuntu one

Subscribers

People subscribed via source and target branches