Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 163256 - [PATCH] app-portage/portage-utils: portage ROOT support in qmerge
Summary: [PATCH] app-portage/portage-utils: portage ROOT support in qmerge
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Tools (show other bugs)
Hardware: All Linux
: High enhancement
Assignee: Portage Utils Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-01-22 15:28 UTC by Natanael Copa
Modified: 2007-01-23 15:58 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
qmerge_portroot_support.patch (qmerge_portroot_support.patch,6.52 KB, patch)
2007-01-22 15:29 UTC, Natanael Copa
Details | Diff
qmerge_portroot_support_v2.patch (qmerge_portroot_support_v2.patch,6.52 KB, patch)
2007-01-22 18:45 UTC, Natanael Copa
Details | Diff
qmerge_portroot_support_v3.patch (qmerge_portroot_support_v3.patch,6.53 KB, patch)
2007-01-23 07:28 UTC, Natanael Copa
Details | Diff
qmerge-mkdirhier.diff (qmerge-mkdirhier.diff,1.29 KB, patch)
2007-01-23 15:18 UTC, solar (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Natanael Copa 2007-01-22 15:28:38 UTC
Added support for portage ROOT in qmerge applet.

Reproducible: Always




I had problems with compiling the cvs version but 0.1.23 seems to be unmodified so the patch is agains 0.1.23.
Comment 1 Natanael Copa 2007-01-22 15:29:11 UTC
Created attachment 107799 [details, diff]
qmerge_portroot_support.patch
Comment 2 solar (RETIRED) gentoo-dev 2007-01-22 17:10:16 UTC
Natanael,
thanks for the patch but could you please. 

cd portage-utils/tests/source
make check

# then attach an updated patch. tia
Comment 3 Natanael Copa 2007-01-22 18:45:43 UTC
Created attachment 107813 [details, diff]
qmerge_portroot_support_v2.patch

Updated patch attached.

btw... Why is BUFSIZE better than PATH_MAX? Looks like its double the size than needed for path operations.
Comment 4 solar (RETIRED) gentoo-dev 2007-01-22 21:43:57 UTC
(In reply to comment #3)
> Created an attachment (id=107813) [edit]
> qmerge_portroot_support_v2.patch
> 
> Updated patch attached.
> 
> btw... Why is BUFSIZE better than PATH_MAX? Looks like its double the 
> size than needed for path operations.

You want _Q_PATH_MAX 

main.c

/* make sure our buffers are as big as they can be */
#if PATH_MAX > _POSIX_PATH_MAX  /* _Q_PATH_MAX */
# define _Q_PATH_MAX PATH_MAX
#else
# define _Q_PATH_MAX _POSIX_PATH_MAX
#endif


Comment 5 Natanael Copa 2007-01-23 07:28:35 UTC
Created attachment 107875 [details, diff]
qmerge_portroot_support_v3.patch

s/BUFSIZE/_Q_PATH_MAX/g

thanks!
Comment 6 solar (RETIRED) gentoo-dev 2007-01-23 15:18:30 UTC
Created attachment 107898 [details, diff]
qmerge-mkdirhier.diff

Commited v3 of your patch. I would like to make the following change to it to avoid the extra system() call. Could you please test? tia
Comment 7 Natanael Copa 2007-01-23 15:32:38 UTC
(In reply to comment #6)
> Created an attachment (id=107898) [edit]
> qmerge-mkdirhier.diff
> 
> Commited v3 of your patch. I would like to make the following change to it to
> avoid the extra system() call. Could you please test? tia
> 

Yes. it looks better. works for me.

btw... seems like empty dirs are not properly removed with qmerge -U. I think it can be commited to cvs anyway. Its a good start.
Comment 8 solar (RETIRED) gentoo-dev 2007-01-23 15:58:21 UTC
(In reply to comment #7)

> btw... seems like empty dirs are not properly removed with qmerge -U.

Feel free to poke around. 
A blind rmdir() as long as no other pkg owns said dir is safe.

---------------------------

Closing bug as FIXED. 
Any other additions to qmerge just file a new bug. Or search for bugs 
assigned to portage-utils@gentoo if you want to take a stab at any of the 
existing requests.