Haggis Veteran Posted October 28, 2013 Veteran Share Posted October 28, 2013 Added a small update that you can get from the git :) Quote The Gnome version would only pick up three digits i.e 3.4.2 if for example the version was 3.4.2.1 the version would not be picked up resulting in an error, change implemented so that it will pick up full version number no matter how long. Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596058717 Share on other sites More sharing options...
Art_X Posted October 28, 2013 Share Posted October 28, 2013 did my update get added too? :shifty: Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596058721 Share on other sites More sharing options...
Haggis Veteran Posted October 28, 2013 Veteran Share Posted October 28, 2013 On 28/10/2013 at 19:11, Demonic Darkness said: did my update get added too? :shifty: lol not yet was just a very small bug fix first :P Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596058809 Share on other sites More sharing options...
Art_X Posted October 28, 2013 Share Posted October 28, 2013 boooo :rofl: Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596059047 Share on other sites More sharing options...
Haggis Veteran Posted October 29, 2013 Veteran Share Posted October 29, 2013 I dont know how to package yet so will need Orangekiller to do that but i will try get gentoo installed tonight to test too question for you is there like a snapshot i can install or do i have to compile everything even in a VM? Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596059795 Share on other sites More sharing options...
Art_X Posted October 29, 2013 Share Posted October 29, 2013 you could use the livecd to test it out, dont think they have any snapshots Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596059849 Share on other sites More sharing options...
Art_X Posted October 29, 2013 Share Posted October 29, 2013 Just to show it works, well does here, only thing you will need to compile is perl-Switch by issuing - emerge perl-Switch Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596060061 Share on other sites More sharing options...
Haggis Veteran Posted October 29, 2013 Veteran Share Posted October 29, 2013 cool i will have a look later :D did not know gentoo had a live cd Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596060163 Share on other sites More sharing options...
Art_X Posted October 29, 2013 Share Posted October 29, 2013 On 29/10/2013 at 12:22, Haggis said: cool i will have a look later :D did not know gentoo had a live cd http://www.gentoo.org/main/en/where.xml Gentoo 20121221 LiveDVD (released December 21, 2012) (up to 3.7 gigabytes depending on arch) amd64 x86 Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596060173 Share on other sites More sharing options...
Karl L. Posted October 29, 2013 Author Share Posted October 29, 2013 Sorry for the late reply. I will try to find some time to review the Gentoo support patch later today. I have been busy with another project recently. Off topic rant: The libparted documentation needs some serious improvement! I have to read the libparted source to understand almost everything. And sometimes even that doesn't make the intended use of each function obvious, so I have to read the parted front-end source too. The parted source would even be helpful if there wasn't so much indirection to initialize each Ped* struct. Also, why does the libparted source include a healthy amount of Doxygen documentation but no doxy file to generate a reference manual? I have been a little hesitant to incorporate official support for more distributions into Stats because it makes new releases increasingly difficult to sufficiently test (without hundreds of gigs of virtual machines, that is). Therefore I have been (slowly) working on a proper integration testing framework to drastically reduce the support burden of testing new releases. It is currently far from complete (therefore I have not pushed its development branch to GitHub, not to mention merged it into master), but it will hopefully appear in a future release. The diagnostic reports Stats 2.x can generate are extremely helpful for implementing this framework. FizzyWidget, would you mind generating a diagnostic report for me on your Gentoo system? I would greatly appreciate it. Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596060579 Share on other sites More sharing options...
Gerowen Posted October 29, 2013 Share Posted October 29, 2013 Just an FYI, since having the issue with the funky window borders and it screenshotting minimized windows, I've upgraded from Ubuntu 13.04 to Xubuntu 13.10, and the issue doesn't seem to exist any more. Not sure if it was the switch from Ubuntu to Xubuntu, the upgrade from 13.04 to 13.10, or what, but it's working great for me now, :-) Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596060609 Share on other sites More sharing options...
Art_X Posted October 29, 2013 Share Posted October 29, 2013 On 29/10/2013 at 15:34, xorangekiller said: Sorry for the late reply. I will try to find some time to review the Gentoo support patch later today. I have been busy with another project recently. Off topic rant: The libparted documentation needs some serious improvement! I have to read the libparted source to understand almost everything. And sometimes even that doesn't make the intended use of each function obvious, so I have to read the parted front-end source too. The parted source would even be helpful if there wasn't so much indirection to initialize each Ped* struct. Also, why does the libparted source include a healthy amount of Doxygen documentation but no doxy file to generate a reference manual? I have been a little hesitant to incorporate official support for more distributions into Stats because it makes new releases increasingly difficult to sufficiently test (without hundreds of gigs of virtual machines, that is). Therefore I have been (slowly) working on a proper integration testing framework to drastically reduce the support burden of testing new releases. It is currently far from complete (therefore I have not pushed its development branch to GitHub, not to mention merged it into master), but it will hopefully appear in a future release. The diagnostic reports Stats 2.x can generate are extremely helpful for implementing this framework. FizzyWidget, would you mind generating a diagnostic report for me on your Gentoo system? I would greatly appreciate it. I would do if it hadnt crashed and seems to have wiped itself out - and in the mood I am in atm, I am not inclined to put linux back on that laptop any time soon, seems not matter what distro i use i get issue :angry: Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596060617 Share on other sites More sharing options...
Karl L. Posted October 29, 2013 Author Share Posted October 29, 2013 On 29/10/2013 at 15:44, Gerowen said: Just an FYI, since having the issue with the funky window borders and it screenshotting minimized windows, I've upgraded from Ubuntu 13.04 to Xubuntu 13.10, and the issue doesn't seem to exist any more. Not sure if it was the switch from Ubuntu to Xubuntu, the upgrade from 13.04 to 13.10, or what, but it's working great for me now, :-) I had trouble replicating your issue, hence the lack of a commit with a fix for it. I suspect that it is actually a bug in Compiz, not Stats (and by extension ImageMagick::Import or Scrot). Since XFCE does not use Compiz (unlike Unity, which is literally built on it), your switch to Xubuntu seems to support that theory. On 29/10/2013 at 15:45, FizzyWidget said: I would do if it hadnt crashed and seems to have wiped itself out - and in the mood I am in atm, I am not inclined to put linux back on that laptop any time soon, seems not matter what distro i use i get issue :angry: I understand, and I appreciate your contributions regardless. However, Gentoo seems like an odd choice of distro if you want something relatively effortless and stable. If you decide to try again with another distro, I recommend Debian. I can't promise that you will have no issues with it (especially without knowing what problems you had with the distros you previously ran), but it is generally counted as one of the best supported and most stable Linux distributions. Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596060667 Share on other sites More sharing options...
Art_X Posted October 29, 2013 Share Posted October 29, 2013 I have an image i can use to get it back up and working, but a simple ext4 defrag of /home should not cause the kernel to spaz out and take the partition with it - i will see what i can do, I think the laptop is just too new and none of the kernels seem to play nice, so debian using an older kernel may not be an issue for me Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596061339 Share on other sites More sharing options...
Art_X Posted October 30, 2013 Share Posted October 30, 2013 seeing as you asked so nicely - here is the report you wanted :) http://pastebin.com/DbTp2vrT set to expire in a week Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596063295 Share on other sites More sharing options...
Karl L. Posted October 30, 2013 Author Share Posted October 30, 2013 On 30/10/2013 at 15:58, FizzyWidget said: seeing as you asked so nicely - here is the report you wanted :) http://pastebin.com/DbTp2vrT set to expire in a week Thank you. I downloaded it. I'm sure it will help. Sorry I didn't have time to review your patch yesterday. I likely won't have time tonight either, but I will try to get to it by this weekend. Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596064635 Share on other sites More sharing options...
Art_X Posted October 30, 2013 Share Posted October 30, 2013 there was something i found when i put the clean image back on that will need to be added to the readme should you decide to implement the changes Required applications for gentoo (should be on a working system) If not please run emerge and make sure you have the following installed lsb_release xrandr xdpyinfo perl-Switcher scrot If any of the packages above are not installed, then run emerge lsb_release xrandr xdpyinfo perl-Switcher scrot Seeing as Gentoo decides to have files in different places as other distributions you will need to make a symlink in /etc to the hostname file in /etc/conf.d/hostname This is done by using ln -s /etc/conf.d/hostname /etc/hostname Should I have some time tomorrow I will try and sort that out and upload a newer version Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596064671 Share on other sites More sharing options...
Karl L. Posted October 30, 2013 Author Share Posted October 30, 2013 On 30/10/2013 at 23:07, FizzyWidget said: Should I have some time tomorrow I will try and sort that out and upload a newer version If you could provide a diff of your changes that would be fantastic. A diff would make it much easier for me to review the changes (and import them into Git); otherwise I have to handle the merger myself when I commit the changes to the repository. If you make a copy of the pristine source directory before making changes, you can generate a clean diff as follows: $ diff -rupN haggis-stats-orig/ haggis-stats/ > stats-add-gentoo-support-v1.patch Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596064769 Share on other sites More sharing options...
Art_X Posted October 30, 2013 Share Posted October 30, 2013 Will do, having an issue atm due to the fact that Gentoo likes to have the hostname file say hostname='name_Here' and i dont know enough of perl to get it to remove it without causing issues elsewhere. Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596064781 Share on other sites More sharing options...
Art_X Posted October 31, 2013 Share Posted October 31, 2013 One patch as requested, it may seem large but i think thats because when you look at the patch it thinks all your files are from 1970, and everything needs to be replaced :D http://pastebin.com/9CJB0FGV - Will expire in 1 week I still cant get it to not show the host name as Hostname: Hostname='name' as I say I am not that great in perl, everything else seems to work fine though :) Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596066065 Share on other sites More sharing options...
Karl L. Posted October 31, 2013 Author Share Posted October 31, 2013 On 31/10/2013 at 12:31, FizzyWidget said: One patch as requested, it may seem large but i think thats because when you look at the patch it thinks all your files are from 1970, and everything needs to be replaced :D http://pastebin.com/9CJB0FGV - Will expire in 1 week I still cant get it to not show the host name as Hostname: Hostname='name' as I say I am not that great in perl, everything else seems to work fine though :) That diff is very noisy, and I think I know why. It is likely that your text editor munged with the spacing and/or line endings. Therefore virtually every line appears to be new whereas it really is not. Although the improper dates are certainly indicative of... something, I don't think they are the source of the problem since diff only captures substantive changes. I can work with the diff you provided as-is if necessary, but would you be willing to produce it again with --ignore-space-change, --ignore-tab-expansion, and --strip-trailing-cr? I would greatly appreciate it if that produces a cleaner diff. The modified command is below. $ diff -rupNEb --strip-trailing-cr haggis-stats-orig/ haggis-stats/ > stats-add-gentoo-support-v1.patch Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596066277 Share on other sites More sharing options...
Art_X Posted October 31, 2013 Share Posted October 31, 2013 http://pastebin.com/FNnd0ciM Seems even with the new line I was getting the same large patch - re-downloading the source again and then running it through nano seems to have cleared it up :D Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596066383 Share on other sites More sharing options...
Karl L. Posted October 31, 2013 Author Share Posted October 31, 2013 The latest version of your patch is much nicer. Thank you FizzyWidget; you have been very accommodating. I will commit it when I get back from work today. Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596066433 Share on other sites More sharing options...
Art_X Posted October 31, 2013 Share Posted October 31, 2013 Gentto support may be very short lived, if it locks up on me one more time tonight I will give debian 7.2 a try, and if it happens on that its back to windows :/ Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596067681 Share on other sites More sharing options...
Karl L. Posted November 1, 2013 Author Share Posted November 1, 2013 FizzyWidget, I just committed official Gentoo support to the Stats repository. You can check it out here. It is based the latest version of your patch, which I enhanced it further to integrate better with the existing Stats codebase. I also added support for the Gentoo hostname file to the script rather than merely mentioning a workaround in the documentation. I am not ready to do another release just yet, but you are welcome to check out the latest commit in the Stats repository. We try to keep the master branch in working condition at all times. You can either download a zip of master generated by GitHub or clone the repository to try it. $ git clone https://github.com/xorangekiller/haggis-stats.git Link to comment https://www.neowin.net/forum/topic/1179351-linux-system-stats-script-version-221-released/page/3/#findComment-596068135 Share on other sites More sharing options...
Recommended Posts