Puppy Linux Discussion Forum Forum Index Puppy Linux Discussion Forum
Puppy HOME page : puppylinux.com
"THE" alternative forum : puppylinux.info
 
 FAQFAQ   SearchSearch   MemberlistMemberlist   UsergroupsUsergroups   RegisterRegister 
 ProfileProfile   Log in to check your private messagesLog in to check your private messages   Log inLog in 

The time now is Sun 21 Dec 2014, 20:06
All times are UTC - 4
 Forum index » Advanced Topics » Puppy Projects
Woof at Github
Moderators: Flash, JohnMurga
Post_new_topic   Reply_to_topic View_previous_topic :: View_next_topic
Page 2 of 14 Posts_count   Goto page: Previous 1, 2, 3, 4, ..., 12, 13, 14 Next
Author Message
Karl Godt


Joined: 20 Jun 2010
Posts: 3982
Location: Kiel,Germany

PostPosted: Wed 13 Nov 2013, 18:15    Post_subject:  

Don't know what I am doing :

/root/KarlGodtWOOF.GIT/woof-CE/woof-code/rootfs-skeleton/etc/rc.d/rc.shutdown
Code:
#130217 01micko: relocate umount of network shares.

PATH=/bin:/sbin:/usr/bin:/usr/sbin
#read PUPPYVERSION </etc/puppyversion

#110923
. /usr/bin/gettext.sh # enables use of eval_gettext (several named variables) and ngettext (plurals)
export TEXTDOMAIN=rc.shutdown
export OUTPUT_CHARSET=UTF-8

. /etc/rc.d/functions4puppy #v3.01
#variables created at bootup by init script in initrd...
. /etc/rc.d/PUPSTATE #v2.02
. /etc/DISTRO_SPECS #v412
ORIGPUPMODE=$PUPMODE #v2.22

if [ "$PUPMODE" =  5 ] ; then
clear
exec 1> /dev/null 2>&1
fi

which rfkill &>/dev/null && rfkill unblock all #110919 jemimah has this in fluppy. if don't do this, may be locked at next boot.

#w482 use xorriso if it is installed (see also functions4puppy)...
CDRECORD='cdrecord'
MKISOFS='mkisofs'
if [ "`which xorriso`" != "" ];then
 CDRECORD='xorrecord'
 MKISOFS='xorrisofs' #growisofs reads this variable.
fi
export CDRECORD MKISOFS

#echo "`eval_gettext \"\\\${DISTRO_NAME} is now shutting down...\"`" > /dev/console
#echo "${DISTRO_NAME} is now shutting down..." > /dev/console
pidof poweroff 2>/dev/null && {
echo "${DISTRO_NAME} ${DISTRO_VERSION} $(gettext 'is now shutting down...')" > /dev/console #120130
} || {
pidof reboot 2>/dev/null && {
echo "${DISTRO_NAME} ${DISTRO_VERSION} $(gettext 'is now rebooting...')" > /dev/console
} || {
#echo $(gettext "Executing /etc/rc.d/rc.shutdown...")
echo "Executing /etc/rc.d/rc.shutdown..."
}; }

#100315 improper shutdown check. see /etc/rc.d/rc.sysinit, /init in initramfs, and /sbin/init...
if [ -f /fsckme.flg ];then
 if [ -f /tmp/dmesg_e2fsck_warnings1 ];then #120717 rc.sysinit creates this file for full installation.
  if [ "`grep -G "(${PDEV1})" /tmp/dmesg_e2fsck_warnings1`" = "" ];then
   rm -f /fsckme.flg
  else
   echo -n 'MAXIMAL' >> /fsckme.flg #read by /sbin/initNEW
  fi
 else
  rm -f /fsckme.flg
 fi
fi
[ -f /initrd${PUP_HOME}/fsckme.flg ] && rm -f /initrd${PUP_HOME}/fsckme.flg

#puppy.sfs is in a subdirectory, default to saving in it...
PUPSFSFILE=`echo "$PUPSFS" | cut -f 3 -d ','`
PSUBDIR=`dirname "$PUPSFSFILE"`
[ "$PSUBDIR" = "/" ] && PSUBDIR=""
[ $PUPMODE -eq 5 -a "$DEV1FS" = "iso9660" ] && PSUBDIR="" #100920 booting off cd, always default have savefile at /.
[ $PUPMODE -eq 5 -a "$DEV1FS" = "udf" ] && PSUBDIR=""     #130128 ditto, for udf f.s.
[ "$DEV1FS" = "msdos" ] && DEV1FS="vfat" #110919

#100920 reverse...
##100915 if booting from cd, .sfs's now optionally copied into a subdir, also want save-file in it...
## (warning: puppy.sfs on cd must be at /)
#if [ "$PSUBDIR" = "" ];then
# if [ "$DEV1FS" = "iso9660" ];then #test if booted off cd.
#   PSUBDIR="`echo -n "${DISTRO_IDSTRING}" | cut -c 1-8`" #limit dir name to 8 chars.
# fi
#fi

#for a full hd install, only setting PUPMODE (=2). no /initrd, no initrd.gz.

killzombies() {

And much more to figure out ..

_________________
«Give me GUI or Death» -- I give you [[Xx]term[inal]] [[Cc]on[s][ole]] .
Macpup user since 2010 on full installations.
People who want problems with Puppy boot frugal Razz
Back to top
View user's profile Send_private_message Visit_website 
Karl Godt


Joined: 20 Jun 2010
Posts: 3982
Location: Kiel,Germany

PostPosted: Wed 13 Nov 2013, 19:07    Post_subject:  

Looks quite ok now with

/usr/git/bin/git commit -am "01 adjustments to rc.shutdown"

git push origin

https://github.com/KarlGodt/woof-CE/commit/1cb63e26014728e403e327f3474f20e31aafff76

not finished, but am really happy !!
Very Happy
Back to top
View user's profile Send_private_message Visit_website 
mavrothal


Joined: 24 Aug 2009
Posts: 1900

PostPosted: Thu 14 Nov 2013, 03:16    Post_subject:  

Karl Godt wrote:
git push origin

https://github.com/KarlGodt/woof-CE/commit/1cb63e26014728e403e327f3474f20e31aafff76


Congratulations Karl! Very Happy
You managed to find and solve all strange issues with git and made the first fork. Cool

Let me take this opportunity though to reiterate some things that I just mentioned at the bottom of the first post.

I'm sure many of us have seen BK's commits in his fossil repo, where many things are changing in one commit and some are not even mentioned in the commit message.
But this was perfectly OK because this was/is his own repo and a message was more of a landmark (besides most of his scripts have their "internal" versioning with comments on the top of the file).

This practice can not work in a collaborative project and may even generate some unnecessary friction if/when pull requests will be asked to be modified.
So with the best intentions, let me use your commit as an example of things that we should not be doing (that I learned the hard way Crying or Very sad ).

The commit message "01 adjustments to rc.shutdown" tells nothing about what was done and why.
In this case there is not even a second line to explain a bit more about it.
Any change in a file could go under "adjustments", "changes" or "fixes", but this is obvious since you change the file.
A more precise and verbal commit message makes it also easier to track it down from the git log. If for example you X is not turning off you can grep for shutdown and X and see if you can find a relevant commit

The second problem is the plural.
We should not do more than one kind of change in one commit.
If you change more things ie "Do not clear console during shutdown" and "bring calls to external variables relier" and "kill X only if it is running during shutdown" etc. you should make more commits.
This way if 1 out of the 5 things that are changed generates a problem will be easy to isolate and resolve.
It is not really the size of the commit but the functions that is affecting that define the number of commits needed.
One commit can change 200 lines and the other can be a single letter.
If you put them together makes it really hard to isolate issues related to the single letter change.
Git also has the "blame" option that shows which commit is the last one for each line of code.
If a problem in a line is identified and this belongs to a "multi purpose" commit makes it harder to determine what a consecutive change will affect.

Also a reasoning behind a change is always helpful.
For example you want to make shutdown more verbal so potential problems can be spotted and solved, which is fine, but what about making this thing optional, with a "debug" argument for example.
Puppy has a "quiet" boot why "verbal" shutdown?
Which brings us back to the "multi commit".
If it is broken down to the different things that is doing and there is an objection about the default verbal shutdown, the other commits could be pulled "as is" and the specific one be revised.
Actually the FOSS practice is, when a major change involves changing many things/functions to be presented as a series of patches instead a big one, ie "Allow alternative WM; remove rox hard codding", "Allow alternative WM; call WM from /etc/profile", "Allow alternative WM;..." etc so it is clear that are going together but is also easy to follow and improve as/if needed.

I'm really happy about your input and certainly this post is not intended as criticism of any kind.
I just felt that I should clarify few things so our life can be easier down the road and move more efficiently. Wink
It may look like unnecessary "formalism" but when many people are involved in a project, is vital.

PS: and after all the preaching (Razz ), you may want to revert your commit and reimplemented in smaller and more concise chunks, so it can be easier to pull when/if you ask.

Edited_times_total
Back to top
View user's profile Send_private_message 
Deacon


Joined: 19 Mar 2013
Posts: 65

PostPosted: Thu 14 Nov 2013, 04:05    Post_subject:  

IT LIVES!

You're working on Woof, so Puppy LIVES!

I haven't the foggiest idea what you people are doing but there's a new Woof trunk to work with! I think that is what it's called! Hurrah!
Back to top
View user's profile Send_private_message 
wanderer

Joined: 20 Oct 2007
Posts: 230

PostPosted: Thu 14 Nov 2013, 05:06    Post_subject:  

thanks mavrothal

this is the way to go

wanderer
Back to top
View user's profile Send_private_message 
Karl Godt


Joined: 20 Jun 2010
Posts: 3982
Location: Kiel,Germany

PostPosted: Thu 14 Nov 2013, 08:28    Post_subject:  

mavrothal, you are absolutely right !

Me too was confused seeing that much red and green at once .

I have set geany as default editor and my geany is set to handle whitespace/tabs for example .
I think that it needs to start saving the file unedited in the editor , just to convert space and tab first and then commit that as info on formatting superfluous space and newlines .

The short info message was also because of being unsure , if the shell could handle multiple lines .

For now , I need to figure out , which parts of git on my systems need really fixes to work with github .
I have the installation similar on four partitions on the current machine .

Code:
[user]
   name = Karl Godt
   email = karlgodt@excite.de
[credential]
   helper = cache --timeout=3600
[core]
   editor = /usr/bin/geany
[http]
   sslVerify = false

The above root/.gitconfig file of mine, created by the git global commandos, still irritates me about where to find the git ca cert file . Firefox I remember, occasionally offers to download some when visiting web pages .

And I am still confused about whether to have the need for a webbrowser github access is needed , or if git from commandline can handle everything , like creating forks .

I am happy, not to have brought the github server down with my commit - but stability is one of Linux/GNU strength , we all know .
Back to top
View user's profile Send_private_message Visit_website 
Karl Godt


Joined: 20 Jun 2010
Posts: 3982
Location: Kiel,Germany

PostPosted: Thu 14 Nov 2013, 10:10    Post_subject:  

I have now reverted the commit on my local repo that I copied from the partition 5 to partition 6 but it seems not to made it into my fork on the github server automatically :
Code:
bash-3.00# git reset --hard HEAD^
HEAD is now at 96ee655 Update README

see here :http://stackoverflow.com/questions/927358/how-to-undo-the-last-git-commit

Note : I use my "normal" git now without recompiling anything :
Code:
bash-3.00# git --version
git version 1.6.4.1

It seems that sslVerify = false does the trick(s) ..
Back to top
View user's profile Send_private_message Visit_website 
Iguleder


Joined: 11 Aug 2009
Posts: 1927
Location: Israel, somewhere in the beautiful desert

PostPosted: Thu 14 Nov 2013, 16:29    Post_subject:  

Made my first contribution.

IMHO, the next logical step is to get rid of xorgwizard, that crappy udev replacement shell script, legacy code for ancient kernels (i.e everything below kernel 2.6.32.x) and all the Xorg 7.3 legacy stuff.

_________________
My homepage
My GitHub profile
Back to top
View user's profile Send_private_message Visit_website MSNM 
ICQ 
zigbert


Joined: 29 Mar 2006
Posts: 5814
Location: Valåmoen, Norway

PostPosted: Thu 14 Nov 2013, 17:14    Post_subject:  

mavrothal wrote:
We should not do more than one kind of change in one commit.
I understand fully this, but it also makes it impossible to improve guis. - Like this Pmount variant. (It has been default in 2 Slacko releases, so should be well tested by now.)

Barry has never focused on the gui-side, and I could gently let my fingers swipe over some gui-code.

But if woof-CE is meant to be kept in the Barry-tradition, I will continue to focus on my own apps.


Sigmund

_________________
Stardust resources
Back to top
View user's profile Send_private_message Visit_website 
Iguleder


Joined: 11 Aug 2009
Posts: 1927
Location: Israel, somewhere in the beautiful desert

PostPosted: Thu 14 Nov 2013, 17:24    Post_subject:  

I think applications like Pmusic should be merged into woof-CE's rootfs-skeleton. They're part of any Puppy, so we can only benefit from the unification of Puppy's code into a single repository.

Is there any reason not to do this? Smile

_________________
My homepage
My GitHub profile
Back to top
View user's profile Send_private_message Visit_website MSNM 
ICQ 
01micko


Joined: 11 Oct 2008
Posts: 7841
Location: qld

PostPosted: Thu 14 Nov 2013, 17:52    Post_subject:  

Iguleder wrote:
I think applications like Pmusic should be merged into woof-CE's rootfs-skeleton. They're part of any Puppy, so we can only benefit from the unification of Puppy's code into a single repository.

Is there any reason not to do this? Smile

Heavy development of Pmusic I think. Ziggy is pretty active. Plus some like to woof with deadbeef, aqualung etc instead of Pmusic or even with out ffmpeg which is Pmusic's core..

EDIT: on the other side, ziggy might like it in the git repo and it could be made optional.

_________________
Woof Mailing List | keep the faith Cool |
Back to top
View user's profile Send_private_message Visit_website 
Iguleder


Joined: 11 Aug 2009
Posts: 1927
Location: Israel, somewhere in the beautiful desert

PostPosted: Thu 14 Nov 2013, 19:09    Post_subject:  

Got good ol' Xfbdev to work with today's GCC and x86_64. Porting such a big pile of ancient code to x86_64 wasn't easy at all Laughing

With JWM and a kernel, it's possible to assemble 50 MB Puppy (uncompressed!) without any applications, with all core X fonts in. Not bad at all! I'll try to build a static GTK1 and gtkdialog - maybe I can make woof-CE support such a minimalistic Puppy, so we can finally have an "official" truly modular Puppy, in the spirit of Tiny Core (a small bootable ISO with a graphical desktop and graphical package management).

_________________
My homepage
My GitHub profile
Back to top
View user's profile Send_private_message Visit_website MSNM 
ICQ 
musher0


Joined: 04 Jan 2009
Posts: 4465
Location: Gatineau (Qc), Canada

PostPosted: Thu 14 Nov 2013, 19:23    Post_subject:  

Hello, all.

This is an excellent initiative! It will certainly improve Puppy building
methodology, as mentioned by mavrothal in his founding post.

I wish a long and fruitful life to the Puppy git hub.

BFN.

musher0

_________________
"Logical entities must not be multiplied needlessly." / "Il ne faut pas multiplier les êtres logiques inutilement." (Ockham)
Back to top
View user's profile Send_private_message Visit_website 
Karl Godt


Joined: 20 Jun 2010
Posts: 3982
Location: Kiel,Germany

PostPosted: Thu 14 Nov 2013, 19:44    Post_subject:  

No comments to the recent posts from my side.
Have now pushed a new commit about rc.shutdown , that only strips trailing spaces :
https://github.com/KarlGodt/woof-CE/commit/a7c01ab53768434d0fc41d96c508fa172f03857f
My first commit is still there as "rc.shutdown-01goal" and will be removed , then when it is transferred into the rc.shutdown file piece by piece .

Was having problems with my older git when pushing ,
and searching the net did not reveal much except that older git were apparently having problems when pushing, so used my new git for that .
I needed to
Code:
/usr/git/bin/git push --verbose --force origin

use the --force option though, and have no clue why simple push failed .

The shell was working well on multiple lines for the commit message .
Back to top
View user's profile Send_private_message Visit_website 
01micko


Joined: 11 Oct 2008
Posts: 7841
Location: qld

PostPosted: Thu 14 Nov 2013, 21:24    Post_subject: woof-CE mailing list  

Ha what a bout a good old fashioned mailing list Smile

This will be a cool place for all those following woof development but not particularly interested in getting their hands dirty and it will be great for developer collaboration as well..

http://woof-ce.26403.n7.nabble.com/Welcome-to-the-woof-mailing-list-td2.html

_________________
Woof Mailing List | keep the faith Cool |
Back to top
View user's profile Send_private_message Visit_website 
Display_posts:   Sort by:   
Page 2 of 14 Posts_count   Goto page: Previous 1, 2, 3, 4, ..., 12, 13, 14 Next
Post_new_topic   Reply_to_topic View_previous_topic :: View_next_topic
 Forum index » Advanced Topics » Puppy Projects
Jump to:  

Rules_post_cannot
Rules_reply_cannot
Rules_edit_cannot
Rules_delete_cannot
Rules_vote_cannot
You cannot attach files in this forum
You can download files in this forum


Powered by phpBB © 2001, 2005 phpBB Group
[ Time: 0.1100s ][ Queries: 12 (0.0049s) ][ GZIP on ]