00:09 | staffencasa has joined IRC (staffencasa!~staffenca@8-220.ptpg.oregonstate.edu) | |
00:21 | coffeecoco has left IRC (coffeecoco!~root@unaffiliated/coffeecoco, Ping timeout: 260 seconds) | |
00:35 | coffeecoco has joined IRC (coffeecoco!~root@unaffiliated/coffeecoco) | |
00:47 | vagrantc has left IRC (vagrantc!~vagrant@freegeek/vagrantc, Quit: leaving) | |
01:06 | bennabiy_away is now known as bennabiy | |
01:11 | coffeecoco has left IRC (coffeecoco!~root@unaffiliated/coffeecoco, Ping timeout: 264 seconds) | |
01:50 | coffeecoco has joined IRC (coffeecoco!~root@unaffiliated/coffeecoco) | |
02:09 | coffeecoco has left IRC (coffeecoco!~root@unaffiliated/coffeecoco, Ping timeout: 264 seconds) | |
02:20 | coffeecoco has joined IRC (coffeecoco!~root@unaffiliated/coffeecoco) | |
02:32 | |Paradox| has joined IRC (|Paradox|!~iamparado@c-71-206-130-134.hsd1.va.comcast.net) | |
02:39 | work_alkisg has left IRC (work_alkisg!~alkisg@plinet.ioa.sch.gr, Quit: Leaving.) | |
02:44 | coffeecoco has left IRC (coffeecoco!~root@unaffiliated/coffeecoco, Ping timeout: 264 seconds) | |
02:51 | coffeecoco has joined IRC (coffeecoco!~root@unaffiliated/coffeecoco) | |
03:36 | vagrantc has joined IRC (vagrantc!~vagrant@freegeek/vagrantc) | |
03:41 | monkwitdafunk has joined IRC (monkwitdafunk!~AndChat49@24.114.36.100) | |
03:41 | monkwitdafunk has joined IRC (monkwitdafunk!~AndChat49@24.114.36.100) | |
03:42 | monkwitdafunk has joined IRC (monkwitdafunk!~AndChat49@24.114.36.100) | |
03:42 | monkwitdafunk has joined IRC (monkwitdafunk!~AndChat49@24.114.36.100) | |
03:43 | coffeecoco has left IRC (coffeecoco!~root@unaffiliated/coffeecoco, Ping timeout: 245 seconds) | |
04:03 | coffeecoco has joined IRC (coffeecoco!~root@unaffiliated/coffeecoco) | |
04:09 | coffeecoco has left IRC (coffeecoco!~root@unaffiliated/coffeecoco, Ping timeout: 245 seconds) | |
04:17 | biophoton has left IRC (biophoton!~andy@h35.209.22.98.dynamic.ip.windstream.net, Ping timeout: 264 seconds) | |
04:37 | freedomrun has joined IRC (freedomrun!~freedomru@unaffiliated/freedomrun) | |
04:38 | coffeecoco has joined IRC (coffeecoco!~root@unaffiliated/coffeecoco) | |
04:38 | <bennabiy> vagrantc: I just resubmitted my merge request
| |
04:38 | night!
| |
04:40 | <vagrantc> bennabiy: great :)
| |
05:01 | work_alkisg has joined IRC (work_alkisg!~alkisg@plinet.ioa.sch.gr) | |
05:01 | work_alkisg is now known as alkisg | |
05:01 | <alkisg> Good morning
| |
05:48 | khildin has joined IRC (khildin!~khildin@ip-213-49-85-183.dsl.scarlet.be) | |
06:07 | jeje has joined IRC (jeje!c400126e@gateway/web/freenode/ip.196.0.18.110) | |
06:07 | <jeje> I am still figuring out how I can boot HP t410 AiO thin client to Ubuntu LTSP server
| |
06:08 | anyone ever configured this before
| |
06:08 | gbaman has joined IRC (gbaman!~gbaman@host81-130-119-33.in-addr.btopenworld.com) | |
06:10 | <alkisg> ARM-based? You'd need an ARM chroot with a loader etc etc...
| |
06:10 | Maybe it'd be easier to install x2go on it as an alternative to the included rdp/ica...
| |
06:12 | mikkel_ has joined IRC (mikkel_!~mikkel@80-199-146-42-static.dk.customer.tdc.net) | |
06:13 | <jeje> yes it is ARM BASED Built using a high performance Texas Instruments TMS320DM8148 ARM® Cortex processor with 1 GB DDR3 SDRAM and 2GB eMMC Flash memory capacity.
| |
06:15 | well not so sure about installing any additional agent on it I could check the x2go option
| |
06:15 | @alkisg kindly substantiate this
| |
06:17 | <vagrantc> those look like they'd make decent thin clients...
| |
06:18 | but booting LTSP on arm is an adventure
| |
06:21 | <alkisg> vagrantc: have you looked at bennabiy's merge request? does the Debian/ and Ubuntu/ approach seem acceptable to you, or would you prefer a dedicated LinuxMint/ folder? (it would need a few hacks to be implemented that way, but it's doable...)
| |
06:22 | The .diff was quite small with the Debian/ & Ubuntu/ approach...
| |
06:23 | <vagrantc> alkisg: haven't looked yet, but that seems clean enough
| |
06:28 | hm.
| |
06:28 | seems like detect_vendor needs to be mangled anyways... :(
| |
06:29 | <jeje> interesting
| |
06:30 | so essential in your opinion, I shouldnt be looking at LTSP then, probably look at Linux RDP??
| |
06:31 | <vagrantc> jeje: depends on how much work you want to do
| |
06:32 | jeje: in theory, they could work as genuine LTSP thin clients... but it's quite probably a lot of work
| |
06:33 | jeje: i did explore the HP t5325 as an LTSP thin client ... no idea how similar the hardware is.
| |
06:33 | jeje: http://cascadia.debian.net/trenza/Documentation/HP_t5325_Debian_Ltsp_Howto/
| |
06:46 | <jeje> ok
| |
06:46 | I would like to use them as thin clients but am flexible
| |
06:47 | there are about 120 terminals that we have to deploy in the students lab environment
| |
06:48 | vagrantc all I need at this point is to be able to use them with any open source system out there, its the reason I went for Ubuntu
| |
06:50 | <vagrantc> using their RDP client and xrdp on your ubuntu server is probably the easiest to at least get something working
| |
06:53 | <jeje> ok
| |
06:56 | have you got a guide on implementing the xrdp on the ubuntu server
| |
06:56 | <vagrantc> apt-get install xrdp
| |
06:57 | <jeje> alright
| |
06:57 | <vagrantc> then point your rdp client at the server ... don't recall needing to do much more than that
| |
06:58 | <jeje> ok
| |
06:58 | vagrantc I was looking at the previous documentation that you sent me
| |
06:59 | the t5325 are similar and I think that this should be able to work but I will look at both options
| |
06:59 | <vagrantc> it's a question of weather or not you can find a workable kernel
| |
06:59 | and how to actually load it
| |
07:08 | <jeje> alright
| |
07:11 | vagrantc has left IRC (vagrantc!~vagrant@freegeek/vagrantc, Quit: leaving) | |
07:18 | gbaman has left IRC (gbaman!~gbaman@host81-130-119-33.in-addr.btopenworld.com, Remote host closed the connection) | |
07:19 | gbaman has joined IRC (gbaman!~gbaman@host81-130-119-33.in-addr.btopenworld.com) | |
07:20 | <jeje> ok cool I will try this out and get back to you
| |
07:23 | gbaman has left IRC (gbaman!~gbaman@host81-130-119-33.in-addr.btopenworld.com, Remote host closed the connection) | |
07:38 | jeje has left IRC (jeje!c400126e@gateway/web/freenode/ip.196.0.18.110, Ping timeout: 250 seconds) | |
07:41 | mikkel_ has left IRC (mikkel_!~mikkel@80-199-146-42-static.dk.customer.tdc.net, Quit: Leaving) | |
07:58 | gbaman has joined IRC (gbaman!~gbaman@dab-bhx1-h-12-3.dab.02.net) | |
08:00 | gbaman has left IRC (gbaman!~gbaman@dab-bhx1-h-12-3.dab.02.net, Remote host closed the connection) | |
08:03 | bennabiy has left IRC (bennabiy!~Thunderbi@24-181-55-79.dhcp.gnvl.sc.charter.com, Ping timeout: 248 seconds) | |
08:05 | bennabiy has joined IRC (bennabiy!~Thunderbi@24-181-55-79.dhcp.gnvl.sc.charter.com) | |
08:23 | alkisg is now known as work_alkisg | |
08:27 | ogra_ has left IRC (ogra_!~ogra_@p5098ed03.dip0.t-ipconnect.de, Ping timeout: 264 seconds) | |
08:28 | ogra_ has joined IRC (ogra_!~ogra_@p5098ed03.dip0.t-ipconnect.de) | |
08:32 | MrOnion_ has joined IRC (MrOnion_!~petero@62.172.154.18) | |
08:37 | gbaman has joined IRC (gbaman!~gbaman@dab-bhx1-h-58-3.dab.02.net) | |
08:42 | gbaman has left IRC (gbaman!~gbaman@dab-bhx1-h-58-3.dab.02.net, Ping timeout: 256 seconds) | |
09:26 | freedomrun has left IRC (freedomrun!~freedomru@unaffiliated/freedomrun, Quit: So long and thanks for all the fish) | |
09:35 | bobby_C has joined IRC (bobby_C!~bobby@85-124-22-227.teleworker.xdsl-line.inode.at) | |
09:43 | biophoton has joined IRC (biophoton!~andy@h35.209.22.98.dynamic.ip.windstream.net) | |
09:51 | Phantomas has joined IRC (Phantomas!~Phantomas@ubuntu/member/phantomas) | |
09:55 | JustSomeGuy has left IRC (JustSomeGuy!~JustSomeG@port-87-193-233-146.static.qsc.de, Quit: Verlassend) | |
10:20 | workingcats has left IRC (workingcats!~workingca@85.232.30.129, Quit: Leaving) | |
10:22 | Phantomas has left IRC (Phantomas!~Phantomas@ubuntu/member/phantomas, Quit: Leaving.) | |
10:38 | khildin has left IRC (khildin!~khildin@ip-213-49-85-183.dsl.scarlet.be, Ping timeout: 245 seconds) | |
10:51 | khildin has joined IRC (khildin!~khildin@ip-213-49-85-183.dsl.scarlet.be) | |
10:53 | freedomrun has joined IRC (freedomrun!~freedomru@unaffiliated/freedomrun) | |
11:15 | biophoton has left IRC (biophoton!~andy@h35.209.22.98.dynamic.ip.windstream.net, Ping timeout: 264 seconds) | |
11:19 | alkisg has joined IRC (alkisg!~alkisg@ubuntu/member/alkisg) | |
11:27 | mgariepy has left IRC (mgariepy!mgariepy@ubuntu/member/mgariepy, Ping timeout: 264 seconds) | |
11:28 | alkisg has left IRC (alkisg!~alkisg@ubuntu/member/alkisg, Quit: Leaving.) | |
11:40 | mgariepy has joined IRC (mgariepy!mgariepy@ubuntu/member/mgariepy) | |
11:42 | bobby_C has left IRC (bobby_C!~bobby@85-124-22-227.teleworker.xdsl-line.inode.at, Ping timeout: 252 seconds) | |
12:02 | alkisg has joined IRC (alkisg!~alkisg@ubuntu/member/alkisg) | |
12:16 | coffeecoco has left IRC (coffeecoco!~root@unaffiliated/coffeecoco, Disconnected by services) | |
12:16 | coffeecoco has joined IRC (coffeecoco!~root@unaffiliated/coffeecoco) | |
12:16 | Phantomas has joined IRC (Phantomas!~Phantomas@ubuntu/member/phantomas) | |
12:17 | coffeecoco1 has joined IRC (coffeecoco1!~root@2403:4800:100:ff3a:2e41:38ff:fe90:8258) | |
12:18 | willianmazzardo has joined IRC (willianmazzardo!~textual@187.53.208.106) | |
12:21 | brunolambert has joined IRC (brunolambert!brunolambe@nat/revolutionlinux/x-zecspfvsgglbocqa) | |
12:23 | freedomrun has left IRC (freedomrun!~freedomru@unaffiliated/freedomrun, Quit: So long and thanks for all the fish) | |
12:28 | alkisg has left IRC (alkisg!~alkisg@ubuntu/member/alkisg, Quit: Leaving.) | |
12:40 | <bennabiy> alkisg and vagrantc: Hold off on the merge. The detect vendor hack is not working properly yet.
| |
12:43 | alkisg and vagrantc: It seems that the variable set in detect_vendor() is not getting it to the right folder. I wonder if perhaps having one folder for LinuxMint and then having the individual scripts that need special resources to do the checks for debian vs ubuntu
| |
12:52 | dsugar100 has joined IRC (dsugar100!~dsugar@columbia.tresys.com) | |
12:54 | workingcats has joined IRC (workingcats!~workingca@212.122.48.77) | |
12:57 | lifeboy has left IRC (lifeboy!~roland@105-236-50-112.access.mtnbusiness.co.za, Ping timeout: 240 seconds) | |
13:20 | <bennabiy> If I want to see how the code is setting the variables, would the proper format be debug=ltsp-build-client ltsp-build-client ?
| |
13:25 | !bennabiy-todo
| |
13:25 | <ltsp`> bennabiy-todo: Get this working...
| |
13:57 | administrator has joined IRC (administrator!be56de6d@gateway/web/freenode/ip.190.86.222.109) | |
13:57 | administrator is now known as Guest48926 | |
13:58 | <Guest48926> alguien me puede ayudar con epoptes
| |
13:59 | <bennabiy> tu hablas ingles?
| |
14:04 | <Guest48926> no
| |
14:04 | solo español
| |
14:05 | Guest48926 has left IRC (Guest48926!be56de6d@gateway/web/freenode/ip.190.86.222.109) | |
14:06 | brunolambert has left IRC (brunolambert!brunolambe@nat/revolutionlinux/x-zecspfvsgglbocqa, Ping timeout: 264 seconds) | |
14:06 | <bennabiy> !learn LinuxMint as Now Available: https://launchpad.net/~bennabiy/+archive/ltsp-ppa Developmental Branch for Mint support (Does not support LMDE yet, sorry) We are currently working on getting changes made upstream, but as a temporary fix, this is available. The client build process might be a bit unstable, as I am actively making changes to it, but everything else is supported upstream.
| |
14:06 | <ltsp`> The operation succeeded.
| |
14:19 | brunolambert has joined IRC (brunolambert!brunolambe@nat/revolutionlinux/x-bohkvghrcnjnzolo) | |
14:21 | khildin has left IRC (khildin!~khildin@ip-213-49-85-183.dsl.scarlet.be, Remote host closed the connection) | |
14:21 | khildin has joined IRC (khildin!~khildin@ip-213-49-85-183.dsl.scarlet.be) | |
14:23 | bennabiy has left IRC (bennabiy!~Thunderbi@24-181-55-79.dhcp.gnvl.sc.charter.com, Ping timeout: 248 seconds) | |
14:24 | bennabiy has joined IRC (bennabiy!~Thunderbi@24-181-55-79.dhcp.gnvl.sc.charter.com) | |
14:50 | brunolambert has left IRC (brunolambert!brunolambe@nat/revolutionlinux/x-bohkvghrcnjnzolo, Ping timeout: 245 seconds) | |
14:50 | vagrantc has joined IRC (vagrantc!~vagrant@freegeek/vagrantc) | |
15:02 | <bennabiy> vagrantc: I am uploading my latest changes. I will resubmit for review based on current state
| |
15:02 | I fixed the vendor_detect() and a few other little bugs, but it seems to be building fine on my test machine through my ppa
| |
15:03 | Also, I made an entry for LinuxMint for any who need access to testing support
| |
15:03 | !LinuxMint
| |
15:03 | <ltsp`> LinuxMint: (#1) "Coming soon - to get something working right away, follow these steps before building your client. #1) sudo ln -s /usr/share/ltsp/plugins/ltsp-build-client/Ubuntu /usr/share/ltsp/plugins/ltsp-build-client/LinuxMint #2) sudo ltsp-build-client --arch=i386 --dist=(UbuntuEquivDistribution), or (#2) "Coming soon - to get something working right away, follow these steps before building your client. #1) sudo ln (2 more messages)
| |
15:03 | brunolambert has joined IRC (brunolambert!brunolambe@nat/revolutionlinux/x-ccqcqqlrkttronup) | |
15:04 | <bennabiy> Hmm, it did not get my latest update
| |
15:05 | !unlearn LinuxMint
| |
15:05 | <ltsp`> Error: "unlearn" is not a valid command.
| |
15:08 | <vagrantc> bennabiy: it adds more, you need to tell it to forget the older entries
| |
15:08 | !vagrant-todo
| |
15:08 | <ltsp`> vagrant-todo: (#1) pxemenus, or (#2) tweak debian/rules to install appropriate update-kernels.conf, or (#3) update-kernels fix memtest support or drop it
| |
15:08 | <vagrantc> learn vagrant-todo as forget this one
| |
15:08 | <bennabiy> !forget LinuxMint
| |
15:08 | <ltsp`> Error: 3 factoids have that key. Please specify which one to remove, or use * to designate all of them.
| |
15:08 | <vagrantc> !learn vagrant-todo as forget this one
| |
15:08 | <ltsp`> The operation succeeded.
| |
15:09 | <vagrantc> !vagrant-todo
| |
15:09 | <ltsp`> vagrant-todo: (#1) pxemenus, or (#2) tweak debian/rules to install appropriate update-kernels.conf, or (#3) update-kernels fix memtest support or drop it, or (#4) forget this one
| |
15:09 | <vagrantc> ltsp`: forget vagrant-todo 4
| |
15:09 | <bennabiy> !forget LinuxMint 1 2
| |
15:09 | <ltsp`> Error: There is no such factoid.
| |
15:10 | <vagrantc> something like that
| |
15:10 | <bennabiy> !LinuxMint
| |
15:10 | <ltsp`> LinuxMint: (#1) "Coming soon - to get something working right away, follow these steps before building your client. #1) sudo ln -s /usr/share/ltsp/plugins/ltsp-build-client/Ubuntu /usr/share/ltsp/plugins/ltsp-build-client/LinuxMint #2) sudo ltsp-build-client --arch=i386 --dist=(UbuntuEquivDistribution), or (#2) "Coming soon - to get something working right away, follow these steps before building your client. #1) sudo ln (2 more messages)
| |
15:10 | <bennabiy> !forget LinuxMint 1
| |
15:10 | <ltsp`> The operation succeeded.
| |
15:10 | <vagrantc> !forget vagrant-todo 4
| |
15:10 | <ltsp`> The operation succeeded.
| |
15:10 | <vagrantc> !vagrant-todo
| |
15:10 | <ltsp`> vagrant-todo: (#1) pxemenus, or (#2) tweak debian/rules to install appropriate update-kernels.conf, or (#3) update-kernels fix memtest support or drop it
| |
15:10 | <bennabiy> !forget LinuxMint 1
| |
15:10 | <ltsp`> The operation succeeded.
| |
15:11 | <bennabiy> !LinuxMint
| |
15:11 | <ltsp`> LinuxMint: Now Available: https://launchpad.net/~bennabiy/+archive/ltsp-ppa Developmental Branch for Mint support (Does not support LMDE yet, sorry) We are currently working on getting changes made upstream, but as a temporary fix, this is available. The client build process might be a bit unstable, as I am actively making changes to it, but everything else is supported upstream.
| |
15:12 | <vagrantc> bennabiy: one thing that's made your patches hard to review is whitespace inconsistancies
| |
15:13 | bennabiy: and it seems like some changes are whitespace-only...
| |
15:13 | <bennabiy> Look at revision I just pushed up
| |
15:13 | <vagrantc> ok
| |
15:15 | <bennabiy> The main files changed compared to upstream are:
| |
15:15 | Modified (5)
| |
15:15 |  common/ltsp-common-functions
| |
15:15 |  server/share/ltsp/plugins/ltsp-build-client/Debian/000-basic-configuration
| |
15:15 |  server/share/ltsp/plugins/ltsp-build-client/Ubuntu/000-basic-configuration
| |
15:15 |  server/share/ltsp/plugins/ltsp-build-client/Ubuntu/010-updates-mirrors
| |
15:15 |  server/share/ltsp/plugins/ltsp-build-client/Ubuntu/030-artwork
| |
15:15 | The debian side could have more done to it, but that should be enough to at least make a build suceed
| |
15:16 | !forget bennabiy-todo *
| |
15:16 | <ltsp`> The operation succeeded.
| |
15:18 | <vagrantc> bennabiy: and i would put your case statement inside of the if [ -e /etc/ltsp/sysconfig ...
| |
15:18 | in the else section
| |
15:20 | bennabiy: and then if CURVEND = LinuxMint ... echo Debian or Ubuntu, otherwise echo CURVEND
| |
15:21 | <bennabiy> That would work.
| |
15:22 | <vagrantc> bennabiy: similarly, in basic-configuration ... just use the existing DIST and check for linuxmint, and then change/add settings f linuxmint
| |
15:23 | bennabiy: then you don't need the else clause at all
| |
15:23 | and makes for a smaller diff
| |
15:23 | <bennabiy> The problem is mint needs both dists
| |
15:24 | <vagrantc> sure, and there's a basic-configuration for each
| |
15:24 | <bennabiy> It relies on both ubuntu dist, which is not obtained from lsb_release
| |
15:24 | <vagrantc> bennabiy: also, i wouldn't touch debian yet :p
| |
15:24 | bennabiy: just makes for a bigger diff... make one feature at a time
| |
15:25 | <bennabiy> The way alkisg was saying, made it sound like I had to have some support for both, or none at all
| |
15:25 | so I was doing at least the basics
| |
15:26 | <vagrantc> maybe alkisg and i are giving you conflicting advice
| |
15:27 | <bennabiy> He ultimately deferred to you about the choice
| |
15:27 | I am totally happy to not even touch the debian side of it, until I need to
| |
15:27 | <vagrantc> bennabiy: for the first review, i'd just do ubuntu
| |
15:27 | once we have a clear idea of what code is needed, then add debian
| |
15:27 | <bennabiy> ok
| |
15:28 | <vagrantc> still a lot of whitespace confusing bits...
| |
15:30 | bennabiy: so in basic config, i would leave DIST as is ... if [ lsb_release --id --short = linuxmint ]; then mintdist = dist ; dist = ...
| |
15:31 | bennabiy: do you really need mint-meta-mate in a thin client?
| |
15:31 | bennabiy: seems like that should only be added to fatclients
| |
15:32 | bennabiy: and what's mint-meta-core?
| |
15:33 | <bennabiy> mint-meta-core is the basics for making a mint chroot according to the devs
| |
15:34 | <vagrantc> bennabiy: with update-mirrors, only put the mint stuff in you if statement, no need for the else clause.
| |
15:34 | bennabiy: if mintdist ; the add_mirrors mintfoo ; fi
| |
15:34 | bennabiy: ad_mirrors regular mirrors
| |
15:36 | minimize code duplication that way
| |
15:36 | bennabiy: the atkwork setting seems a little overcomplicated...
| |
15:37 | bennabiy: you could probably just add mate-desktop to the ist of desktops to check for
| |
15:38 | someone might have mate-desktop installed without mint, for example
| |
15:38 | <bennabiy> Like I said, I do not use fat clients, nor have I ever, so it is a foreign concept to me
| |
15:39 | GrembleBean has joined IRC (GrembleBean!~Ben@cpc35-aztw23-2-0-cust207.18-1.cable.virginmedia.com) | |
15:40 | <vagrantc> bennabiy: could even leave t out for the first pass
| |
15:40 | <bennabiy> thank you!
| |
15:41 | If I get into fat clients, I might revisit it, but I am not going to set up a fat client to support it
| |
15:41 | <vagrantc> get the core functionality merged
| |
15:41 | <bennabiy> Any support is better than no support :)
| |
15:41 | yes
| |
15:42 | <vagrantc> bennabiy: with all that tidied up, should be good to go
| |
15:47 | although i ponder weather debian/ubuntu shouldn't just carry the detect_vendor in it's own vendor-functions...
| |
15:47 | as well as the sysdist stuff
| |
15:57 | <bennabiy> which dist uses sysdist?
| |
15:57 | RH?
| |
16:08 | ok, pushing most recent changes
| |
16:09 | MrOnion_ has left IRC (MrOnion_!~petero@62.172.154.18, Quit: MrOnion_) | |
16:10 | <bennabiy> vagrantc: should be up now
| |
16:14 | <vagrantc> bennabiy: yeah, the fedora family, although i think the latest RH stuff doesn't
| |
16:16 | <bennabiy> ok, I tried making your change to 030-artwork, but I would never know if it breaks anything
| |
16:16 | It should be pushed now
| |
16:17 | 2494
| |
16:17 | <vagrantc> bennabiy: still a lot of needless whitespace changes, but coming along... :)
| |
16:18 | <bennabiy> vagrantc: I am not seeing what you are saying about whitespace changes. are you comparing to ltsp trunk, or my last files?
| |
16:19 | <vagrantc> https://code.launchpad.net/~bennabiy/ltsp/ltsp-LinuxMint/+merge/187442
| |
16:20 | bennabiy: in the debian basic-configuration ...
| |
16:20 | that highlights the problem, but it's all over the place
| |
16:20 | <bennabiy> how can I revert a file back to original ?
| |
16:21 | <vagrantc> bennabiy: are you using tabs instead of spaces?
| |
16:21 | <bennabiy> yes
| |
16:21 | <vagrantc> that might be you issue, then
| |
16:21 | <bennabiy> I noticed that some was tabbed, some was spaced, so I used tabs
| |
16:21 | <vagrantc> bennabiy: i wouldn't touch the artwork plugin
| |
16:21 | <bennabiy> ok
| |
16:22 | <vagrantc> bennabiy: use whatever keeps the diff minimal :)
| |
16:28 | bennabiy: thre's a stray fi;; in detect_vendor at the end... should just be ;;
| |
16:28 | * vagrantc also refers ;; on it's own line | |
16:28 | <vagrantc> prefers
| |
16:30 | might actually be simler to use a nested if staement rather than the case statement, too
| |
16:32 | bennabiy: later
| |
16:32 | * vagrantc waves | |
16:32 | vagrantc has left IRC (vagrantc!~vagrant@freegeek/vagrantc, Quit: leaving) | |
16:34 | bobby_C has joined IRC (bobby_C!~bobby@85-124-22-227.teleworker.xdsl-line.inode.at) | |
16:42 | willianmazzardo has left IRC (willianmazzardo!~textual@187.53.208.106, Quit: Computer has gone to sleep.) | |
16:57 | willianmazzardo has joined IRC (willianmazzardo!~textual@187.53.208.106) | |
17:09 | Phantomas1 has joined IRC (Phantomas1!~Phantomas@ubuntu/member/phantomas) | |
17:10 | Phantomas has left IRC (Phantomas!~Phantomas@ubuntu/member/phantomas, Ping timeout: 248 seconds) | |
17:18 | GrembleBean has left IRC (GrembleBean!~Ben@cpc35-aztw23-2-0-cust207.18-1.cable.virginmedia.com, Quit: I Leave) | |
17:20 | Phantomas has joined IRC (Phantomas!~Phantomas@ubuntu/member/phantomas) | |
17:21 | Phantomas1 has left IRC (Phantomas1!~Phantomas@ubuntu/member/phantomas, Ping timeout: 240 seconds) | |
17:38 | bobby_C has left IRC (bobby_C!~bobby@85-124-22-227.teleworker.xdsl-line.inode.at, Ping timeout: 256 seconds) | |
17:38 | vagrantc has joined IRC (vagrantc!~vagrant@freegeek/vagrantc) | |
17:45 | bobby_C has joined IRC (bobby_C!~bobby@85-124-22-227.teleworker.xdsl-line.inode.at) | |
17:46 | <bennabiy> vagrantc: That should be it...
| |
18:03 | <vagrantc> bennabiy: one more whitespace in the updates-mirrors plugin
| |
18:03 | n
| |
18:04 | bennabiy: the indentation in basic-configuration looks off...
| |
18:05 | bennabiy: and what's mint-meta-core actually pull in?
| |
18:06 | bennabiy: mintsources.conf contains the ubuntu release it's based on?
| |
18:07 | <bennabiy> yes
| |
18:08 | I looked at the whitespace in the updates-mirrors, and cannot see a difference, it is spaced to where it shows on the diff
| |
18:08 | I took out all tabs
| |
18:09 | mint-meta-core pulls in the plymouth artwork as well as a few other basics, to de-ubuntu the client
| |
18:09 | <vagrantc> bennabiy: probably a tabs vs. saces issue
| |
18:10 | <bennabiy> That is what I was saying, there were some tabs used, and then some spaces, and then some tabs and spaces in the same line
| |
18:10 | <vagrantc> somehow it's different
| |
18:10 | <bennabiy> The ;; lines up.
| |
18:10 | It is all spaces now.
| |
18:10 | I tried to change it, and then it told me no changes to commit
| |
18:10 | <vagrantc> maybe the original had a tab
| |
18:11 | <bennabiy> I will check the upstream
| |
18:12 | It is exactly the same.
| |
18:13 | <vagrantc> there's something different
| |
18:14 | bennabiy: maybe trailing whitespace?
| |
18:15 | <bennabiy> I am pushing for 2500 :)
| |
18:15 | try now.
| |
18:15 | 2497
| |
18:17 | vagrantc: how do I get debian to be an option to build for in my ppa?
| |
18:19 | alkisg has joined IRC (alkisg!~alkisg@ubuntu/member/alkisg) | |
18:19 | <bennabiy> alkisg: welcome!
| |
18:20 | <alkisg> Hi guys
| |
18:20 | <bennabiy> I just pushed a new update which I think should be good for a basic start
| |
18:21 | vagrantc hounded me until I got all my whitespaces right. ;)
| |
18:25 | <vagrantc> bennabiy: launchpad doesn't support debian builds
| |
18:25 | <bennabiy> I was afraid of that... I was hoping to use it as a testbed for lmde...
| |
18:26 | vagrantc: I appreciate you not wanting sloppy code. Please don't think I am having a hard time with you (I realize my comment could lend itself towards that)
| |
18:27 | I really appreciate the labors that go into making LTSP what it is
| |
18:29 | <vagrantc> glad you see it that way :)
| |
18:36 | <bennabiy> Does it pass
| |
18:37 | <pscheie> vagrantc, are you coming to BTS this year?
| |
18:37 | <bennabiy> vagrantc: How old are you, if you dont mind me asking?
| |
18:40 | <vagrantc> pscheie: i reall need to
| |
18:41 | bennabiy: i don't bother to keep track
| |
18:42 | <bennabiy> Somehow I thought that would be coming :)
| |
18:42 | <pscheie> vagrantc, well that sounds like a 'yes'!
| |
18:42 | <vagrantc> bennabiy: so, whitespace looks good, but i would indent the code in basic-config and update-mirror
| |
18:43 | bennabiy: just style changes at thiis point, though, but may as well get it
| |
18:44 | <bennabiy> vagrantc: That is encouraging
| |
18:45 | vagrantc: What do you think the indentation on basic-config should be?
| |
18:47 | <vagrantc> bennabiy: everything in your aded if block
| |
18:47 | added
| |
18:48 | <bennabiy> Is the if / else fine? And then indent the added blocks?
| |
18:48 | gbaman has joined IRC (gbaman!~gbaman@host81-130-119-33.in-addr.btopenworld.com) | |
18:49 | <vagrantc> bennabiy: yeah, all the variables should be one indent deeper than the if, and the if should be one indent past the case block
| |
18:49 | bennabiy: so indent it all one further
| |
18:50 | bennabiy: and the add_mirrors line should be indente one
| |
18:50 | sorry for my typing, on a small keyboard
| |
18:51 | <bennabiy> Just make me commit 2 more times :)
| |
18:51 | 2498
| |
18:51 | shoudl be up
| |
18:51 | should*
| |
18:53 | <vagrantc> ok, loks good :)
| |
18:53 | <bennabiy> !vagrantc-todo
| |
18:53 | <ltsp`> vagrantc-todo: Merge LinuxMint changes... (Subliminal)
| |
18:53 | <vagrantc> one last thing... :)
| |
18:54 | <bennabiy> can it be two things?
| |
18:54 | <vagrantc> will probably squash this all into one commit in the end :)
| |
18:56 | <bennabiy> Basically, this is just a starting point, to get basic support there. After that, I might experiment with the Debian side to get more support for it
| |
18:56 | <vagrantc> MINTCOMPONENTS="main upstream import" should use the FOO=${FOO:-"blah blah"} syntax
| |
18:56 | <bennabiy> why?
| |
18:57 | It does not have a commandline implimentation
| |
18:57 | <vagrantc> as well as all the other basic-config vars
| |
18:58 | configuration file
| |
18:58 | <bennabiy> Is that to support someone doing a MINTCOMPONENTS="FOO" ltsp-build-client ?
| |
18:58 | ahh
| |
18:58 | <vagrantc> or that
| |
18:58 | but yes
| |
18:58 | <bennabiy> ok
| |
18:59 | <vagrantc> might be good to add the commandline stuff at some point, too
| |
18:59 | and then we're golden
| |
18:59 | <bennabiy> I wondered if it would be good to have a generic variable setting option where you could do somethign like --var foo --var foo2 etc
| |
18:59 | <alkisg> Fedora: lsb_release ==> command not found
| |
18:59 | <bennabiy> as many as needed in the commandline
| |
19:00 | <alkisg> That's why the "if" was outside of the lsb_release call...
| |
19:00 | So basically ltsp-build-client would fail on fedora with the current detect_vendor
| |
19:00 | <vagrantc> alkisg: but they can add the dependency to pull in lsb release
| |
19:01 | <bennabiy> Ladies and gentlemen... in this corner..
| |
19:01 | <alkisg> vagrantc: sure, but I don't think there's anything to gain by rearranging the if that way
| |
19:01 | <vagrantc> although it's a larger package on fedora, it's not insanely largr
| |
19:01 | alkisg: simplicity?
| |
19:02 | whole oint of lsb is common standards across distros...
| |
19:02 | * bennabiy revisits LinuxMint folder... | |
19:04 | gbaman has left IRC (gbaman!~gbaman@host81-130-119-33.in-addr.btopenworld.com, Remote host closed the connection) | |
19:05 | gbaman has joined IRC (gbaman!~gbaman@host81-130-119-33.in-addr.btopenworld.com) | |
19:05 | <alkisg> vagrantc: I think that if we wanted to simplify the fedora code, it should be a separate commit...
| |
19:06 | bennabiy: also, you call $(lsb_release --id --short | tr " " "_") twice, it'd be better to save its result to a variable
| |
19:06 | (btw don't think I'm strict I don't even use ltsp-build-client at all, if you don't like me proposing things just tell me to stop :))
| |
19:07 | <bennabiy> What do you mean call it twice?
| |
19:07 | <alkisg> I'm looking at https://code.launchpad.net/~bennabiy/ltsp/ltsp-LinuxMint/+merge/183364, at the bottom, the diff,
| |
19:07 | Check line 9 and line 22 there
| |
19:08 | <bennabiy> somehow your version is not up to date with the most recent post. It looks like there is regression
| |
19:09 | <alkisg> Ah let me check the latest version...
| |
19:10 | <bennabiy> wait a second. I am about to commit a few more error.. I means fixes
| |
19:10 | I do have it saved to a variable
| |
19:13 | <alkisg> But you don't reuse the variable later on, so the old call does the same thing again
| |
19:13 | Let me find a good launchpad link to show you what I'm saying...
| |
19:14 | <bennabiy> I just fixed one instance of that... in 000-basic-configuration
| |
19:15 | <alkisg> https://code.launchpad.net/~bennabiy/ltsp/ltsp-LinuxMint/+merge/187442 looks better :)
| |
19:16 | <bennabiy> if [ $CURVEND = "LinuxMint" ]; then
| |
19:16 | MINTDIST=${MINTDIST:-"$(lsb_release -s -c)"}
| |
19:16 | DIST=${DIST:-"$(grep base_codename /usr/share/mintsources/$MINTDIST/mintsources.conf | awk '{split($0,array,"=")} END{print array[2]}')"}
| |
19:17 | <alkisg> if [ -n $MINTDIST ]; then ==> should be "$MINTDIST" there, because if it's empty, then that's equivalent to [ -n ], which is a syntax error, not to [ -n "" ], which isn't a syntax error
| |
19:17 | <bennabiy> The only problem is if someone chooses to build for a different version of mint, it would break
| |
19:17 | <alkisg> In general, it's better to quote more than less
| |
19:17 | <bennabiy> yes
| |
19:18 | <alkisg> if [ $CURVEND = "LinuxMint" ]; then ==> same for "$CURVEND" there
| |
19:18 | About the mirrors and dist etc, would it be possible for someone to specify both MINTDIST and DIST?
| |
19:18 | Would that make sense?
| |
19:19 | <bennabiy> yes
| |
19:19 | If they are running mint
| |
19:19 | that is why I wondered if we should have a command line argument for adding whatever variable
| |
19:19 | <alkisg> Give me a specific example...
| |
19:19 | ..i.e. an example of what distro I have, and what chroot I want to build, that would need both variables
| |
19:20 | <vagrantc> alkisg: yes, the fedora change should happen after talking with the appropriate persons in it's own commit
| |
19:20 | <bennabiy> Mint requires two distributions, and two sets of mirrors,
| |
19:21 | <alkisg> Can I build an olivia chroot with an ubuntu 10.04 mirror?
| |
19:21 | I don't think so, there are specific pairs that can be made there, right?
| |
19:21 | <bennabiy> There are pairs
| |
19:21 | <alkisg> So why use 2 variables?
| |
19:21 | <vagrantc> i think the dist code only handles building the same
| |
19:22 | as the host
| |
19:22 | alkisg: you need both ubuntu and mint dists
| |
19:22 | <alkisg> The pairs are declared in the .conf files
| |
19:23 | <vagrantc> all of them? or only te current ist?
| |
19:23 | dist
| |
19:23 | <bennabiy> When mint first builds the client, it needs to have the ubuntu dist, but then to access the mint mirrors etc, it needs to have the mint dist
| |
19:23 | <alkisg> I think all of them
| |
19:23 | <bennabiy> alkisg: It would break if you wanted to make an older dist on a newer mint
| |
19:23 | <alkisg> Why? It does have the old .conf files
| |
19:23 | <bennabiy> like if I wanted to make a client for precise / maya
| |
19:23 | not on my server it doesnt
| |
19:23 | <vagrantc> could someone paste mintosurces.confto pastebin?
| |
19:23 | <alkisg> It should be enough to state DIST=maya
| |
19:24 | <bennabiy> nope
| |
19:24 | no such thing as maya for debootstrap
| |
19:24 | DIST feeds debootstrap
| |
19:25 | <alkisg> The linuxmint code should split dist into 2 variables
| |
19:25 | <bennabiy> been downthis road
| |
19:25 | <alkisg> But the user shouldn't have to bother with them
| |
19:25 | What if the user says, DIST=10.04 and MINTDIST=Olivia?
| |
19:25 | <vagrantc> ok, i see your point
| |
19:25 | <bennabiy> the user would only have to use both if wanting to do a different version that the server
| |
19:25 | <alkisg> What I'm trying to say is, that MINTDIST shouldn't be a public variable
| |
19:25 | Not exported to the user
| |
19:26 | It should be an internal variable decided by some mint .conf files
| |
19:26 | <vagrantc> they should be able to specify dist, and the mint code should set mintdist basd n that, and reset dist to the appropriate ubuntu value
| |
19:26 | <bennabiy> Where would I put that code?
| |
19:27 | <vagrantc> in basic-config
| |
19:27 | <bennabiy> It seems like I would need to hardcode all the combos using that mindset then
| |
19:27 | <vagrantc> bennabiy: mintsources.conf only contains the running dist?
| |
19:27 | <bennabiy> yes
| |
19:28 | I will pastebin it
| |
19:28 | <vagrantc> also, it should be possible to build mint from ubuntu srver
| |
19:28 | and ubuntu srver won't have mintsources.conf
| |
19:28 | <bennabiy> exactly
| |
19:29 | <vagrantc> bennabiy: so es, you might have to hard-code the pairings in basic-config
| |
19:29 | <bennabiy> in 000-basic-configuration or somewhere else?
| |
19:29 | That would eventually be a lot of pairings?
| |
19:29 | Can we source in a file which would be updated every 6 months?
| |
19:29 | <vagrantc> not if mint gets smarter... :)
| |
19:30 | <bennabiy> I do not see mint "getting smarter" ...
| |
19:30 | <vagrantc> i wouldn't try to future proof it too much
| |
19:30 | include all the pairings you can when you can
| |
19:31 | is the upcoming release usually known in advance?
| |
19:31 | <bennabiy> somewhat
| |
19:31 | <alkisg> bennabiy: /usr/share/mintsources/ has flurry, nadia, olivia etc, are there more?
| |
19:31 | komunista has joined IRC (komunista!~slavko@87.244.209.121) | |
19:33 | <bennabiy> those are all that are in my folder
| |
19:33 | <komunista> hi all
| |
19:33 | <alkisg> Hello
| |
19:33 | bennabiy: I think that folder contains all supported releases, as mentioned in http://www.linuxmint.com/oldreleases.php
| |
19:34 | So it wouldn't be possible to build obsolete releases, ok, but I don't think their repositories would be available either
| |
19:34 | <bennabiy> It doesn't have maya in mine, which is the lts
| |
19:35 | <alkisg> True... I think that's a mistake, but anyway we can't rely on that folder :-/
| |
19:36 | <bennabiy> there is a new release coming out soon as well
| |
19:36 | <alkisg> So yeah either relying on the user providing the "pair", or on hardcoded values
| |
19:36 | ...so, on second thought, I agree with your approach of DIST/MINTDIST
| |
19:37 | <bennabiy> Yes
| |
19:37 | Can you consider the idea of this...
| |
19:37 | <alkisg> Another topic... variables in CAPS are supposed to be "global", needed by other ltsp scripts, but e.g. CURVEND isn't used anywhere else...
| |
19:38 | <bennabiy> ltsp-build-client --arch=i386 --dist=raring --var MINTDIST=olivia --var ...
| |
19:38 | <komunista> after upgared my Debian's LTSP environment to wheezy, I found freenx not working (to connect from teacher's Windows machine), then was success with the X2Go solution
| |
19:38 | <bennabiy> actually it is used in 000-basic-configuration
| |
19:39 | <komunista> please ,is anybody using X2Go on the thinc clients?
| |
19:39 | <bennabiy> I just need to commit the change
| |
19:39 | :)
| |
19:39 | alkisg: I need to pick your brain about epoptes as well :)
| |
19:39 | <komunista> It is possible or documented somewhere?
| |
19:39 | <alkisg> bennabiy: am I looking at the correct diff? https://code.launchpad.net/~bennabiy/ltsp/ltsp-LinuxMint/+merge/187442
| |
19:39 | Ah sorry
| |
19:39 | I just read your previous line ^ :)
| |
19:40 | komunista: using x2go to control a thin client?
| |
19:40 | Or _from_ a thin client to connect to another server?
| |
19:40 | <komunista> to login from thin clients
| |
19:40 | <alkisg> To use x2go instead of LDM?
| |
19:40 | <komunista> alkisg: yes
| |
19:41 | <alkisg> And you need ltsp for netbooting the clients? Or you don't need ltsp at all
| |
19:41 | ?
| |
19:41 | vagrantc has left IRC (vagrantc!~vagrant@freegeek/vagrantc, Ping timeout: 256 seconds) | |
19:41 | vagrantc has joined IRC (vagrantc!~vagrant@freegeek/vagrantc) | |
19:41 | <bennabiy> alkisg: revision 2499
| |
19:41 | <komunista> Yes, I need netboot, because clients has no HDD
| |
19:42 | I tried the NX solution some years ago, it was working but without the guest login, then I abadon this
| |
19:44 | <bennabiy> vagrantc: I made your indents and updated for alkisg's suggestions
| |
19:44 | <alkisg> komunista: x2go has a thin client component, and a login manager component, but I think the best way to do it would be to implement an x2go ltsp screen.d script
| |
19:46 | <bennabiy> alkisg: Is it posisble to use epoptes to control the server and clients from a remote machine?
| |
19:46 | <komunista> alkisg: yes, this was what I think about
| |
19:46 | <alkisg> Yup
| |
19:47 | <komunista> then basicaly it must to work, right?
| |
19:47 | <alkisg> Yup
| |
19:48 | <komunista> OK, thanks I will play with it :-)
| |
19:48 | <alkisg> bennabiy: sed -n 's/base_codename=//p' mintsources.conf
| |
19:50 | Why the _ in MINT_MIRROR? The others, MINTDIST etc don't have one...
| |
19:50 | (maybe it would be better to put one in all of them)
| |
19:50 | <bennabiy> to match EARLY_MIRROR etc
| |
19:51 | I am fine with changing them all to _spacing
| |
19:51 | It would make for commit 2500!
| |
19:51 | Just as long as vagrantc doesn't tell me to change it back :)
| |
19:52 | <alkisg> Hehe :)
| |
19:52 | If you're going to do it, do the same for CUR_VEND
| |
19:52 | <vagrantc> yeah, more _ is good
| |
19:52 | <alkisg> (or CUR_VENDOR)
| |
19:52 | And put more quotes
| |
19:52 | * vagrantc prefers cur_vendor | |
19:52 | <alkisg> case $CURVEND in ==> case "$CURVEND" in etc etc
| |
19:53 | <bennabiy> yes, I still need to fix that one.
| |
19:53 | <alkisg> if [ $(lsb_release -s -c) = "debian" ]; then ==> if [ "$(lsb_release -s -c)" = "debian" ]; then
| |
19:53 | <bennabiy> It is uppercase because I use it later
| |
19:53 | <alkisg> echo $CURVEND ==> echo "$CURVEND"
| |
19:53 | <vagrantc> is CURVEND available in basic-configuration?
| |
19:54 | <alkisg> bennabiy: see an example: CURVEND="*"; cd /; echo $CURVEND ==> bin boot cdrom dev etc home initrd.img lib lost+found ...
| |
19:54 | <vagrantc> or maybe LSB_VENDOR?
| |
19:54 | <bennabiy> alkisg: sed -n 's/base_codename=//p' mintsources.conf brings out raring, when in the directory
| |
19:54 | <vagrantc> since that's what it is?
| |
19:55 | <bennabiy> Ok
| |
19:55 | <alkisg> bennabiy: isn't that what you wanted to print with the awk stuff? I didn't run it...
| |
19:56 | <bennabiy> which is more efficient, sed or awk ?
| |
19:57 | <alkisg> One sed call is more efficient than a grep and an awk calls, but it's more readable too, which is more significant in ltsp-build-client, where speed doesn't matter much
| |
19:58 | (the sed replaces the grep call too)
| |
19:58 | <bennabiy> so do: sed -n 's/base_codename=//p' /usr/share/mintsources/$MINTDIST/mintsources.conf
| |
19:58 | <alkisg> Right, with quotes in the path though :)
| |
19:59 | <bennabiy> err. $MINT_DIST
| |
19:59 | <alkisg> sed -n 's/base_codename=//p' "/usr/share/mintsources/$MINT_DIST/mintsources.conf"
| |
20:00 | It might be worth it to do an if [ -f .../mintsources ] there as well, in case the server doesn't run mint, but you can postpone that part for later :)
| |
20:00 | <bennabiy> would I need to escape the quotes around /usr/shar...
| |
20:00 | since the whole thing is doublequoted
| |
20:01 | <alkisg> No, when you start a $(), inside that it's a different quoting ...universe :)
| |
20:02 | <vagrantc> just when i thought we were don for the moment...
| |
20:02 | <alkisg> Btw if you prefer awk instead of sed (again without grep): awk -F= '/base_codename/{print $2} < mintsourcs.conf
| |
20:02 | <bennabiy> I am fine with sed
| |
20:03 | <vagrantc> and '
| |
20:03 | <alkisg> ...and without < as well :-/
| |
20:04 | <bennabiy> so CURVEND is becoming LSB_VENDOR
| |
20:04 | The rest are just getting spaced _
| |
20:04 | sound right?
| |
20:04 | DIST=${DIST:-"$(sed -n 's/base_codename=//p' "/usr/share/mintsources/$MINT_DIST/mintsources.conf")"}
| |
20:06 | <vagrantc> could use ${MINT_DIST} instead of the extra "
| |
20:06 | <alkisg> vagrantc: no, ${} don't replace quotes
| |
20:06 | They're just eye candy in this case
| |
20:07 | <vagrantc> oh, it works for test -n ${FOO}
| |
20:07 | <alkisg> I think the outside quotes can be safely removed though
| |
20:07 | DIST=${DIST:-$(sed -n 's/base_codename=//p' "/usr/share/mintsources/$MINT_DIST/mintsources.conf")}
| |
20:08 | freedomrun has joined IRC (freedomrun!~freedomru@unaffiliated/freedomrun) | |
20:08 | <bennabiy> Which should I do?
| |
20:08 | * vagrantc agrees | |
20:09 | <alkisg> And a final one from me, case $CURVEND in ==> why not make that a simple if?
| |
20:09 | * bennabiy jaw drops... alkisg and vagrantc agree! | |
20:09 | <alkisg> Haha, we _usually_ agree!!! :)
| |
20:09 | <vagrantc> indeed
| |
20:09 | <bennabiy> I am just a troublemaker
| |
20:10 | <komunista> good night :-)
| |
20:10 | komunista has left IRC (komunista!~slavko@87.244.209.121, Quit: Leaving.) | |
20:10 | <vagrantc> alkisg: yeah, i proposed to make that an iftoo
| |
20:10 | <alkisg> Sorry, didn't see that... cool, let's go for the final merge proposal...!
| |
20:11 | <bennabiy> hmm... so would you do an if , elif to catch all three cases?
| |
20:11 | <vagrantc> alkisg: i gave up asking for it, but since we're here
| |
20:11 | <alkisg> bennabiy: which 3 cases? /me only sees two...
| |
20:12 | <bennabiy> 2 for linuxmint and 1 for others
| |
20:12 | <vagrantc> sysconfig, linuxmint, *
| |
20:12 | <bennabiy> that is why the case
| |
20:12 | <alkisg> bennabiy: no, keep the inside if as it is
| |
20:12 | So, an if inside another if
| |
20:13 | vagrantc +1
| |
20:13 | I just opened ltsp-common-functions and saw the whole picture, I (again) agree with vagrantc
| |
20:14 | * bennabiy is puzzled | |
20:14 | <alkisg> Meh no, it won't work as it is now...
| |
20:14 | Let me pastebin it
| |
20:14 | <bennabiy> let me pastebin what I have, and then I will pastebin what I think you mean
| |
20:17 | http://pastebin.mozilla.org/3147252 As it stands right now
| |
20:17 | <alkisg> http://paste.debian.net/46162/
| |
20:18 | ...with a bit better spacing :)
| |
20:18 | <bennabiy> I am fine with that
| |
20:19 | <vagrantc> yeah, that's what i was thinking... change CUR_VENDOR to LSB_VENDOR though?
| |
20:19 | <bennabiy> What would you recommend the spacing be
| |
20:19 | yes
| |
20:20 | LSB_VENDOR
| |
20:20 | <alkisg> LSB_VENDOR, yup
| |
20:20 | <vagrantc> spacing looks fine to me
| |
20:20 | <alkisg> I meant the identation, a couple of lines are off
| |
20:20 | (of my paste)
| |
20:21 | <vagrantc> 5 spaces instead of 4?
| |
20:22 | <alkisg> lines 8 and 13 in my paste, I think
| |
20:22 | <bennabiy> I will check it
| |
20:23 | <vagrantc> bennabiy is getting good practice at checking for whitespace issues :)
| |
20:23 | <bennabiy> heh
| |
20:23 | whatspace?
| |
20:23 | <alkisg> Erm it seems like a problem in paste.debian.net highlighter
| |
20:23 | <bennabiy> It looks fine on my end,
| |
20:23 | willianmazzardo has left IRC (willianmazzardo!~textual@187.53.208.106, Quit: Computer has gone to sleep.) | |
20:23 | <alkisg> When I look at it at my gedit it's fine, and when I look at it as raw in debian.net it's also fine, but when highlighted, it's off
| |
20:24 | <bennabiy> That was happening to me earlier
| |
20:25 | <alkisg> mozilla highlighter seems better... http://pastebin.mozilla.org/3147309
| |
20:26 | <vagrantc> me apparently didn't see any highlighting
| |
20:26 | <alkisg> It's probably javascript-based
| |
20:27 | ...which we well know you have disabled :D
| |
20:27 | * vagrantc acts surprised | |
20:29 | <bennabiy> heh
| |
20:30 | ok. So I updated the variables to have proper quoting around them, changed to alkisg's code for detect_vendor() made spacing_ changes
| |
20:30 | was there anything else?
| |
20:30 | <alkisg> ...world peace?
| |
20:30 | <bennabiy> I am going for commit 2500!
| |
20:30 | I like that number :)
| |
20:33 | ok pushing changes
| |
20:35 | <vagrantc> looks good.
| |
20:35 | * bennabiy rejoices! | |
20:35 | <vagrantc> now i'd like to figre out how to squash it into a single commit
| |
20:35 | <bennabiy> !vagrantc-todo
| |
20:35 | <ltsp`> vagrantc-todo: Merge LinuxMint changes... (Subliminal)
| |
20:37 | <vagrantc> i wonder if i can approve the merge without actually merging it.
| |
20:37 | khildin has left IRC (khildin!~khildin@ip-213-49-85-183.dsl.scarlet.be, Ping timeout: 246 seconds) | |
20:38 | <vagrantc> haven't used launchpad's merge review at all before
| |
20:38 | also don't have my launchpad login on this machine
| |
20:38 | <alkisg> bennabiy: + if [ "$LSB_VENDOR" = "LinuxMint" ]; then
| |
20:38 | ...do we want to check the "server" there, or the chroot?
| |
20:39 | <bennabiy> in 000-basic-configuration?
| |
20:39 | <alkisg> Yes
| |
20:40 | Suppose I have a mint server and I'm building a plain ubuntu chroot
| |
20:40 | dist=raring or whatever
| |
20:40 | Won't that part of the code set MINT_DIST in that case?
| |
20:40 | <bennabiy> Hmm
| |
20:43 | How would you distinguish between the two?
| |
20:43 | <vagrantc> ouch
| |
20:43 | * alkisg goes to light a cigar and will come back with new thoughts on this... :) | |
20:44 | <bennabiy> How does it work if someone does a dist like RHEL or something?
| |
20:45 | <vagrantc> RH family might be a bit too far off to get reliable results
| |
20:45 | <bennabiy> yes...
| |
20:46 | <vagrantc> but debian dists should be able to build on another
| |
20:46 | <bennabiy> I wonder if there is a way to do a check if DIST is manually set, and if so, not do the Mint Dist
| |
20:46 | <vagrantc> though in practice, it might be a bit buggy
| |
20:46 | <bennabiy> like if I wanted to build an Ubuntu client (which basically, I have been doing in the past to connect to my mint server )
| |
20:46 | <vagrantc> bennabiy: but then what if it's set to a mint dist of a different versin
| |
20:47 | <bennabiy> real world example...
| |
20:47 | <vagrantc> running mint 13, trying to build a mint 12
| |
20:47 | <bennabiy> would fail anyways because 12 is no longer supported
| |
20:48 | * vagrantc is not familiar enough with mint releases | |
20:48 | <vagrantc> current mint release tries to build current release - 1 ... :p
| |
20:49 | or does mint drop support for old releases as soon as a new one is out?
| |
20:49 | <bennabiy> I doubt anyone on ubuntu would *want* to build a mint client on an ubuntu server
| |
20:49 | 1 year
| |
20:49 | unless LTS
| |
20:49 | <alkisg> if [ -n "$MINT_DIST" } || ( [ "$LSB_VENDOR" = "LinuxMint" ] && [ -z "$DIST" ] ) then
| |
20:50 | <vagrantc> that handles building ubuntu on mint, but not building a different mint release on a mint server
| |
20:51 | <bennabiy> can I bring back up the ability to add custom variables to the command line?
| |
20:51 | by --var VARNAME=BLAH
| |
20:51 | <vagrantc> if boolean_is_true WHY_DOES_MINT_MAKE_THIS_SO_HARD ...
| |
20:51 | <bennabiy> or some such?
| |
20:51 | dsugar100 has left IRC (dsugar100!~dsugar@columbia.tresys.com, Quit: dsugar100) | |
20:51 | <bennabiy> hah!
| |
20:52 | <vagrantc> bennabiy: that would require rewriting all the arubmment procssing
| |
20:52 | to support multiple --var foo --var bar
| |
20:52 | <alkisg> (11:50:38 μμ) vagrantc: that handles building ubuntu on mint, but not building a different mint release on a mint server ==> why not?
| |
20:52 | <vagrantc> currently, you'd just end up with bar
| |
20:53 | <alkisg> MINT_DIST="older_version" would still get in the "if"...
| |
20:53 | <vagrantc> alkisg: oh, i see... you'd set MINT_DIST, not DIST
| |
20:53 | <alkisg> You'd set both
| |
20:53 | <vagrantc> sure
| |
20:53 | <alkisg> We want (to compromise with) the following:
| |
20:54 | 1) no params in a mint server => build a mint chroot
| |
20:54 | <vagrantc> (of same version)
| |
20:54 | <alkisg> 2) if one wants to build a mint chroot in a non-mint server, he needs to specify both MINT_DIST and DIST, to provide the "pair"
| |
20:54 | ...in order to avoid us hardcoding all the pairs...
| |
20:55 | <vagrantc> that works
| |
20:55 | <bennabiy> yes
| |
20:55 | <vagrantc> but what if they don't gender identify with the "he" pronoun?
| |
20:56 | <bennabiy> which if someone used a different MINT_DIST version than the server, if it was within the supported dists, it would still work
| |
20:56 | It would just need to be paired with *a* ubuntu version
| |
20:57 | and might not work
| |
20:57 | if not paired properly
| |
20:57 | but someone doing that would probably be aware of the pairings
| |
20:57 | <alkisg> Meh. You English-speaking guys need a neutral pronoun. s/he...
| |
20:57 | <bennabiy> it
| |
20:58 | <vagrantc> alkisg: i've conceeded to use "they"
| |
20:58 | <bennabiy> לךישדגכ
| |
20:58 | <alkisg> If one... they? Meh again
| |
20:59 | <vagrantc> alkisg: it doesn't hurt me as much as s/he
| |
20:59 | * alkisg also hates french guys for using plural for a single person... we greeks picked that up a few decades ago, and it's very annoying | |
20:59 | <vagrantc> alkisg: well, hopefully the hatred doesn't burn too hot :p
| |
21:00 | <alkisg> Haha, no worries, I won't punch anyone in the face over it
| |
21:00 | <bennabiy> alkisg: where would you have me add your line, and would that be enough to get it merged for now, and then I can keep working on fine tuning later?
| |
21:01 | <alkisg> bennabiy: just replace your if with mine
| |
21:01 | <bennabiy> I am about to go for a couple days, and want to make the changes while I am still fresh :)
| |
21:01 | alkisg: does it need a ; before the then?
| |
21:02 | <alkisg> Yeah, and some testing, that it works with mint and doesn't break debian/ubuntu builds
| |
21:02 | Other than that, the merge request looks fine to me too, +1
| |
21:02 | See, 23 lines, not enough to bother with creating 23 symlinks instead of it :D
| |
21:03 | <bennabiy> :P
| |
21:04 | hmm..
| |
21:04 | <vagrantc> finished product may need a few more lines
| |
21:04 | <bennabiy> !bennabiy_todo
| |
21:04 | <ltsp`> Error: "bennabiy_todo" is not a valid command.
| |
21:05 | <bennabiy> If you think of any more changes after I commit this, can you leave them for me?
| |
21:05 | I will check in after the shabbat
| |
21:05 | <vagrantc> sure
| |
21:05 | i'll comment on the mege request
| |
21:05 | <bennabiy> I am going to commit alkisg's suggestion, and then you will have the current changes as they sit
| |
21:05 | * alkisg projected that it would need 60 lines so he'll be fine with a few more later on | |
21:06 | <alkisg> 'night guys
| |
21:06 | <bennabiy> night
| |
21:07 | alkisg has left IRC (alkisg!~alkisg@ubuntu/member/alkisg, Quit: Leaving.) | |
21:08 | <bennabiy> vagrantc: done.
| |
21:09 | <vagrantc> bennabiy: typo in the new code
| |
21:09 | a } should be ]
| |
21:10 | <bennabiy> got it
| |
21:10 | <vagrantc> bennabiy: otherwise it looks merge worthy
| |
21:11 | <bennabiy> pushed
| |
21:11 | _test_
| |
21:12 | <vagrantc> looks good.
| |
21:12 | i'll commit it hen i get to my other computer
| |
21:12 | <bennabiy> great!
| |
21:13 | You are in UK?
| |
21:13 | <vagrantc> due west, over one big pond, stop just shy of the really big pond
| |
21:14 | <bennabiy> Ireland?
| |
21:14 | Dia duit!
| |
21:14 | <vagrantc> no, biger water bodies
| |
21:15 | near the pacific coast on north america
| |
21:15 | a.k.a cascadia
| |
21:16 | <bennabiy> ah,
| |
21:18 | With doug as the flag?
| |
21:19 | I am from Idaho originally, and love the pacific northwest
| |
21:19 | anyways, I will see you when I get back
| |
21:19 | Thanks for all the help!
| |
21:21 | bennabiy is now known as away_bennabiy | |
21:23 | * vagrantc waves | |
21:23 | <vagrantc> a bit late
| |
21:23 | vagrantc has left IRC (vagrantc!~vagrant@freegeek/vagrantc, Quit: leaving) | |
21:46 | bobby_C has left IRC (bobby_C!~bobby@85-124-22-227.teleworker.xdsl-line.inode.at, Ping timeout: 252 seconds) | |
22:01 | gbaman has left IRC (gbaman!~gbaman@host81-130-119-33.in-addr.btopenworld.com, Remote host closed the connection) | |
22:14 | Phantomas1 has joined IRC (Phantomas1!~Phantomas@ubuntu/member/phantomas) | |
22:14 | Phantomas has left IRC (Phantomas!~Phantomas@ubuntu/member/phantomas, Ping timeout: 264 seconds) | |
22:17 | vagrantc has joined IRC (vagrantc!~vagrant@freegeek/vagrantc) | |
22:51 | JustMe_ has joined IRC (JustMe_!d49f71df@gateway/web/freenode/ip.212.159.113.223) | |
22:52 | JustMe_ is now known as SomeoneElse | |
22:52 | SomeoneElse is now known as RandomUnregister | |
22:54 | <RandomUnregister> Working on packaging LTSP for arch w/ a berryterminal client, I've got to the LDM logon screen but then get 'Restarting X and LDM' upon entering any credentials (regardless of whether they exist)
| |
22:54 | Any pointers as to where I've gone wrong please?
| |
22:54 | <vagrantc> RandomUnregister: you need the ssh keys for the server somehow
| |
22:55 | although berryterminal doesn't use LTSP's packaging at all...
| |
22:55 | <RandomUnregister> Thanks, that makes complete sense
| |
22:55 | Nope, it's fun disecting the pieces :)
| |
22:56 | Thanks for the help... ttfn
| |
22:57 | <vagrantc> would be interesting to see arch support added for LTSP...
| |
22:57 | not that i've ever used arch, there are many things i respect about it
| |
23:00 | RandomUnregister has left IRC (RandomUnregister!d49f71df@gateway/web/freenode/ip.212.159.113.223, Ping timeout: 250 seconds) | |
23:03 | Phantomas1 has left IRC (Phantomas1!~Phantomas@ubuntu/member/phantomas, Quit: Leaving.) | |