PATCH: fix undefined value warning [#277239]

Comment	File	Size	Author
#2	modr8-fix-undefined-value-value.patch	440 bytes	eMPee584
	modr8-fix-undefined-value-#value.patch	440 bytes	eMPee584

Comment #1

pwolanin CreditAttribution: pwolanin commented 2 July 2008 at 21:10

can you please re-post the patch without a # in the name? I cannot open the link because of that.

Log in or register to post comments

Comment #2

eMPee584 CreditAttribution: eMPee584 commented 5 July 2008 at 16:09

File	Size
modr8-fix-undefined-value-value.patch	440 bytes

wow that's true sorry.. *g

Log in or register to post comments

Comment #3

NikLP CreditAttribution: NikLP commented 4 September 2008 at 12:05

Is this going anywhere? Is it in dev? HEAD?

Log in or register to post comments

Comment #4

pwolanin CreditAttribution: pwolanin commented 5 September 2008 at 00:46

Status:

Needs review

» Fixed

committed - thanks

Log in or register to post comments

Comment #5

eMPee584 CreditAttribution: eMPee584 commented 7 September 2008 at 12:05

cool thx pwolanin... btw would you mind to also make 6.x-dev releases on each CVS commit? don't know how much work it is but it helps stay on track, f.e. my little update script uses the date of dev tarballs to slurp newest ones off the server..

#!/bin/sh
IFS=''
echo "`date`">update.log
for file in *.tar.gz
do
        cp -p $file $file.tmpcpy
        wget -N --no-proxy http://ftp.drupal.org/files/projects/$file >>update.log 2>&1
        if [[ "$file" -nt "$file.tmpcpy" ]]
                then
                tar -xzf $file
                echo "$file updated"
        fi
        rm $file.tmpcpy
done

Log in or register to post comments

Comment #6

NikLP CreditAttribution: NikLP commented 7 September 2008 at 12:43

Again, there are still no clues as to "where this is fixed" - can someone PLEASE add this information? Lots of people are trying their hardest to use 6.x stuff. Modules are all over the place, still, and lack of this kind of information is just confusing as hell.

Many thanks.

Log in or register to post comments

Comment #7

eMPee584 CreditAttribution: eMPee584 commented 7 September 2008 at 13:15

NikLP don't quite now if i understood you correctly, but if you meant that literally, pwolanin has committed this to CVS HEAD, which is the only place to get it because this module does not yet have a dev release. It's only a silent warning anyways so please calm down ;)