Suggestion and a possible bugfix for 'cb-welcome'

gearge · 2012-05-09 14:16:29

'cb-welcome' script fails to detect internet connection under certain conditions.

Analysis: A component of cb-welcome, '/usr/lib/lib-cb-welcome/check-internet-connection' script, uses ping to check availability of the remote repository. This method is unreliable as ICMP traffic is usually blocked inside corporate networks; though HTTP and FTP access is usually permitted.

Solution: Reimplement the internet connection check, as proposed in the provided script file:
https://www.box.com/s/5935166ff813996cd7f4

Suggestion

The default character repeat rate and delay settings in CB seem much different from defaults provided in most major distributions. As a new convert to CrunchBang I found this somewhat uncomfortable.

"Keyboard Settings" page on CrunchBang wiki describes how to change these settings; though "GUI Method" section needs updating for "Waldorf" release, as xfce-mcs-settings package in no longer available.

I want to suggest adding a section for keyboard configuration to the default Openbox autostart script (/home/user/.config/openbox/autostart). An updated version with reasonable defaults is provided below:
https://www.box.com/s/9c6e37d13e1fad3f0712

The exact values for repeat rate and delay may be adjusted as the community sees fit. What's important is to provided users an option to easily configure these settings (out of the box, so to say). As pling wrote on the wiki, "the developers of Openbox forget about the keyboard". One must agree.

Thanks.

Last edited by gearge (2012-05-09 14:19:38)

rhowaldt · 2012-05-09 14:24:59

see here: http://crunchbanglinux.org/forums/topic … l-scripts/ for the internet connection check.

gearge · 2012-05-09 14:55:52

xaos52 wrote:

Perhaps 'wget' a small file from a debian server and test the return code?

It seems I implemented exactly what xaos52 was suggesting :)

This is a trustworthy method and if it fails, we can honestly assume that (1) no internet connection is available - down or not configured - or (2) that the remote server is down or has some other issues.

I can easily implement a mechanism to allow the user to skip consecutive connection checks, if the first one fails (a simple elif statement should do it), but it wouldn't make sense because of any of the two conditions described above.

Though, it is possible to display different error messages to the user, based on the error code returned by wget. No hassle to implement, BTW.

Last edited by gearge (2012-05-09 14:59:01)

corenominal · 2012-05-10 07:23:27

gearge wrote:

'cb-welcome' script fails to detect internet connection under certain conditions.
Analysis: A component of cb-welcome, '/usr/lib/lib-cb-welcome/check-internet-connection' script, uses ping to check availability of the remote repository. This method is unreliable as ICMP traffic is usually blocked inside corporate networks; though HTTP and FTP access is usually permitted.
Solution: Reimplement the internet connection check, as proposed in the provided script file:
https://www.box.com/s/5935166ff813996cd7f4

cb-welcome is now in my GitHub account, please see: https://github.com/corenominal/cb-welcome

Pull requests welcome smile

gearge · 2012-05-10 20:00:05

Thanks corenominal,

Pull request sent to the original author smile

What about my suggestion about autostart?

corenominal · 2012-05-10 21:08:34

gearge wrote:

Thanks corenominal,
Pull request sent to the original author

Excellent. Pull request accepted, I'll repackage soon. Thank you smile

What about my suggestion about autostart?

Yes, looks good to me and would be a welcome addition. Thank you again smile

gearge · 2012-05-18 19:47:17

Thank you corenominal,

As expected, I also updated relevant pipemenus. Download:
https://www.box.com/s/b9d4cc9708f1dc0620b3

Changes:

* Modified 'connectiontest' function to use reliable wget instead of unreliable ping (as reported previously)
* Moved 'connectiontest' function to a new file (cb-include.cfg), that can be sourced inside any bash script and used to store common variables and functions
* Modified relevant /usr/bin/cb-* scripts to reflect this change
* Modified '/usr/bin/cb-x-www-browser-pipemenu' script to use vendor supplied URLs as primary sources of APT keys; but implemented fallback to packages.crunchbang.org if primary sources fail. This concerns Opera and Google Chrome options.

I'll be happy to fork, if you manage to place /usr/bin/cb-* files on GitHub. :-)

Thanks again.

Last edited by gearge (2012-05-18 19:52:52)

corenominal · 2012-05-18 19:57:39

Excellent, I will get the pipemenu package into my GitHub account. Thank you smile

corenominal · 2012-05-18 20:07:27

corenominal wrote:

Excellent, I will get the pipemenu package into my GitHub account.

All done. smile

z_skie · 2012-05-18 21:05:37

my suggestion is to maybe include some other window managers like awesome or fluxbox, but whatever big_smile

gearge · 2012-05-19 16:52:51

corenominal,

Pull request issued for both 'cb-pipemenus' and 'cb-welcome'.

corenominal · 2012-05-19 17:47:41

Excellent work. Seriously, thank you for your contributions, I really appreciate the help smile

I have accepted the pull requests and will repackage the scripts next week.

CrunchBang Linux

SEARCH

#1 2012-05-09 14:16:29

Suggestion and a possible bugfix for 'cb-welcome'

#2 2012-05-09 14:24:59

Re: Suggestion and a possible bugfix for 'cb-welcome'

#3 2012-05-09 14:55:52

Re: Suggestion and a possible bugfix for 'cb-welcome'

#4 2012-05-10 07:23:27

Re: Suggestion and a possible bugfix for 'cb-welcome'

#5 2012-05-10 20:00:05

Re: Suggestion and a possible bugfix for 'cb-welcome'

#6 2012-05-10 21:08:34

Re: Suggestion and a possible bugfix for 'cb-welcome'

#7 2012-05-18 19:47:17

Re: Suggestion and a possible bugfix for 'cb-welcome'

#8 2012-05-18 19:57:39

Re: Suggestion and a possible bugfix for 'cb-welcome'

#9 2012-05-18 20:07:27

Re: Suggestion and a possible bugfix for 'cb-welcome'

#10 2012-05-18 21:05:37

Re: Suggestion and a possible bugfix for 'cb-welcome'

#11 2012-05-19 16:52:51

Re: Suggestion and a possible bugfix for 'cb-welcome'

#12 2012-05-19 17:47:41

Re: Suggestion and a possible bugfix for 'cb-welcome'

Board footer