Project

General

Profile

Feature #139

Home Inventory

Added by Anonymous about 12 years ago. Updated about 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
04/24/2009
Due date:
% Done:

0%


Description

How many times you look for an object in your home and search into any box, any closet and drawer without find nothing?? Home Inventory is a free, light-weight MySQL/PHP web application aims to quickly organize and find objects.

http://sourceforge.net/projects/homeinv/

History

#1 Updated by Anonymous about 12 years ago

  • Assignee set to mciv

#2 Updated by mciv about 12 years ago

image uploading isn't working after one-click install.

After some digging I found that line 131 of environment.php had an incorrect path statement. It was pointing to htdocs/images/$sDest . Removing htdocs allows images to be uploaded.

Not sure how to correct the one-click installer so that images can be correctly uploaded.

#3 Updated by cpg about 12 years ago

  • Status changed from Assigned to Feedback

hi, i updated the install script to reflect the extra chunk in the patch needed, based on the comments below (i.e. i did not test it because i am remote at the moment).

sending to mciv for feedback. if it works, please close this issue.
if not, please add details.

hope it works! :)

#4 Updated by cpg about 12 years ago

just to share some of the experience, this is what i did.

only one line really needed change, which translates to making a patch for it.

  • went to my webapp dir for homeinv
  • chmod -R 777 . (to let me edit things)
  • saved the original to .new
  • saved the patch to patch-original.diff
  • reverted the html/environment.php to a pristine version
  • changed the .new
  • generated a new .patch

then realized that the `pwd` part, kinda important, because the path of the webapp is NOT guaranteed, needed to be changed.

cd /var/hda/web-apps/homeinv/
chmod -R 777 .
cat > patch-orig.diff # (then copied&pasted the diff with the mouse)
cp html/environment.php html/environment.php.new
patch -p0 -R < patch-orig.diff # now environment.php is pristine as distributed by the authors
vi html/environment.php.new # edited the file and removed htdocs/ per the comments
diff -u -b html/environment.php html/environment.php.new > patch-new.diff # then copy the last chunk back to the install script

the patch-new.diff is the new patch! except that the `pwd` needs to be put back

however, since only one chunk changed (a chunk is the @ ...@ group in the diff)
why touch the rest? simply add the chunk just before the EOF of that patch.

hopefully the above makes sense?

we will do this often with apps. maybe we can develop some apps to do this easier.

#5 Updated by mciv about 12 years ago

I'm understanding more and more what you did to make the install script. Thanks.

Unfortunately the change doesn't change the line in question. I only had a few items in so I removed the app completely and installed using the updated installer but the path still includes htdocs.

#6 Updated by cpg about 12 years ago

done. close if you agree, otherwise assign it back to me.

my bad. the situation is that the second chunk failed to patch:

[22:02:46](1)florence:homeinv$ ll html/environment.php*
-rwxr--r-- 1 apache users 5458 2009-04-25 00:11 html/environment.php
-rwxr--r-- 1 apache users 5407 2007-02-17 02:56 html/environment.php.orig
-rw-r--r-- 1 apache users  385 2009-04-25 00:11 html/environment.php.rej
[22:02:58](1)florence:homeinv$ 

and i was rushing, while i still had battery.

once i looked at the rejected patch (.rej), i realized that there were a lot of $ variables
in the patch.

because the EOF was not 'EOF', substitution of shell variable took place on them. all of them being empty, the patch failed.

you can check it in your .rej file. you will see all $variable were nuked.

solution, because we need the EOF due to the `pwd` substitution in the first patch, i broke it in two
patches, EOF in the first one, 'EOF' in the second one.

#7 Updated by mciv about 12 years ago

  • Status changed from Feedback to Closed

Also available in: Atom