Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 163088 - Please remove RESTRICT="mirror" from all Xorg packages
Summary: Please remove RESTRICT="mirror" from all Xorg packages
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords: Inclusion
Depends on:
Blocks:
 
Reported: 2007-01-21 17:09 UTC by Zhang Le (RETIRED)
Modified: 2007-02-04 18:53 UTC (History)
1 user (show)

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


Attachments
remove RESTRICT="mirror" from Xorg packages' ebuilds (Xorg-RESTRICT-mirror.patch,42.96 KB, patch)
2007-01-22 17:05 UTC, Zhang Le (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zhang Le (RETIRED) gentoo-dev 2007-01-21 17:09:40 UTC
Xorg has many mirrors all over the world. So I think it should be OK for us to mirror all Xorg packages. 
I found that several packages in x11-apps, many packages in x11-libs, x11-proto and media-fonts still have RESTRICT="mirror". I think we can remove 
RESTRICT="mirror" from those ebuilds.
Possible candidates are:
/usr/portage/x11-apps/xcursorgen/xcursorgen-1.0.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-apps/xhost/xhost-1.0.1.ebuild:14:RESTRICT="mirror"
/usr/portage/x11-apps/bdftopcf/bdftopcf-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-apps/sessreg/sessreg-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-apps/sessreg/sessreg-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-apps/appres/appres-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-apps/xsetpointer/xsetpointer-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-apps/xsetpointer/xsetpointer-1.0.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-apps/xdriinfo/xdriinfo-1.0.1.ebuild:11:RESTRICT="mirror"
/usr/portage/x11-apps/xsetmode/xsetmode-1.0.0.ebuild:11:RESTRICT="mirror"
/usr/portage/x11-libs/ViewKlass/ViewKlass-0.7.0.ebuild:9:RESTRICT="nomirror"
/usr/portage/x11-libs/libFS/libFS-1.0.0.ebuild:14:RESTRICT="mirror"
/usr/portage/x11-libs/libSM/libSM-1.0.2.ebuild:14:RESTRICT="mirror"
/usr/portage/x11-libs/libSM/libSM-1.0.1.ebuild:14:RESTRICT="mirror"
/usr/portage/x11-libs/libXp/libXp-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXt/libXt-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXt/libXt-1.0.4.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXrender/libXrender-0.9.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXrender/libXrender-0.9.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXxf86vm/libXxf86vm-1.0.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXres/libXres-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXres/libXres-1.0.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXres/libXres-1.0.3.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libICE/libICE-1.0.2.ebuild:14:RESTRICT="mirror"
/usr/portage/x11-libs/libICE/libICE-1.0.1.ebuild:14:RESTRICT="mirror"
/usr/portage/x11-libs/libX11/libX11-1.0.3.ebuild:14:RESTRICT="mirror"
/usr/portage/x11-libs/libX11/libX11-1.0.1-r1.ebuild:11:RESTRICT="mirror"
/usr/portage/x11-libs/libX11/libX11-1.0.3-r1.ebuild:14:RESTRICT="mirror"
/usr/portage/x11-libs/libXau/libXau-1.0.1.ebuild:11:RESTRICT="mirror"
/usr/portage/x11-libs/libXxf86dga/libXxf86dga-1.0.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/xtrans/xtrans-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/xtrans/xtrans-1.0.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/xtrans/xtrans-1.0.3.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXTrap/libXTrap-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXfixes/libXfixes-4.0.3.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXfixes/libXfixes-4.0.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-libs/libXxf86misc/libXxf86misc-1.0.1.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-misc/3dfb/3dfb-0.6.1.ebuild:10:RESTRICT="nomirror"
/usr/portage/x11-misc/grun/grun-0.9.2.ebuild:12:RESTRICT="nomirror"
/usr/portage/x11-misc/imake/imake-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-misc/seyon/seyon-2.20c.ebuild:10:RESTRICT="nomirror"
/usr/portage/x11-misc/makedepend/makedepend-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xf86dgaproto/xf86dgaproto-2.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/trapproto/trapproto-3.4.3.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/randrproto/randrproto-1.1.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/randrproto/randrproto-1.2.0.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xproxymanagementprotocol/xproxymanagementprotocol-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xf86rushproto/xf86rushproto-1.1.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/fontcacheproto/fontcacheproto-0.1.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/fontsproto/fontsproto-2.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/evieext/evieext-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/renderproto/renderproto-0.9.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/recordproto/recordproto-1.13.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/resourceproto/resourceproto-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xcmiscproto/xcmiscproto-1.1.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xf86bigfontproto/xf86bigfontproto-1.1.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xproto/xproto-7.0.5.ebuild:11:RESTRICT="mirror"
/usr/portage/x11-proto/xproto/xproto-7.0.7.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xproto/xproto-7.0.10.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xproto/xproto-7.0.9.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/videoproto/videoproto-2.2.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/printproto/printproto-1.0.3.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xineramaproto/xineramaproto-1.1.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/dmxproto/dmxproto-2.2.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xextproto/xextproto-7.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xf86miscproto/xf86miscproto-0.9.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/xf86vidmodeproto/xf86vidmodeproto-2.2.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/inputproto/inputproto-1.3.2.ebuild:13:RESTRICT="mirror"
/usr/portage/x11-proto/bigreqsproto/bigreqsproto-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/media-fonts/font-misc-meltho/font-misc-meltho-1.0.0.ebuild:15:RESTRICT="mirror"
/usr/portage/media-fonts/font-arabic-misc/font-arabic-misc-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-xfree86-type1/font-xfree86-type1-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-adobe-100dpi/font-adobe-100dpi-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-adobe-75dpi/font-adobe-75dpi-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-bitstream-100dpi/font-bitstream-100dpi-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/encodings/encodings-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/media-fonts/encodings/encodings-1.0.2.ebuild:13:RESTRICT="mirror"
/usr/portage/media-fonts/font-schumacher-misc/font-schumacher-misc-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-cursor-misc/font-cursor-misc-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-bh-75dpi/font-bh-75dpi-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-misc-misc/font-misc-misc-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-bh-ttf/font-bh-ttf-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/media-fonts/font-ibm-type1/font-ibm-type1-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-bh-type1/font-bh-type1-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-micro-misc/font-micro-misc-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-bitstream-type1/font-bitstream-type1-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-winitzki-cyrillic/font-winitzki-cyrillic-1.0.0.ebuild:12:RESTRICT="mirror"
/usr/portage/media-fonts/font-misc-ethiopic/font-misc-ethiopic-1.0.0.ebuild:13:RESTRICT="mirror"
/usr/portage/media-fonts/font-dec-misc/font-dec-misc-1.0.0.ebuild:12:RESTRICT="mirror"


Reproducible: Always
Comment 1 Zhang Le (RETIRED) gentoo-dev 2007-01-21 17:13:29 UTC
Sorry, please ignore these ebuilds:
/usr/portage/x11-misc/3dfb/3dfb-0.6.1.ebuild:10:RESTRICT="nomirror"
/usr/portage/x11-misc/grun/grun-0.9.2.ebuild:12:RESTRICT="nomirror"
/usr/portage/x11-misc/seyon/seyon-2.20c.ebuild:10:RESTRICT="nomirror"
Comment 2 Donnie Berkholz (RETIRED) gentoo-dev 2007-01-21 22:47:20 UTC
This happened because the upstream digests changed and it was extremely difficult to clean the tarballs off all of our mirrors and get new ones in place.

If you would care to verify that in each case, removing the restriction fetches a valid tarball from a Gentoo mirror, and supply a patch, we can look at making this change.
Comment 3 Zhang Le (RETIRED) gentoo-dev 2007-01-22 05:05:43 UTC
(In reply to comment #2)
> This happened because the upstream digests changed and it was extremely
> difficult to clean the tarballs off all of our mirrors and get new ones in
> place.
you mean upstream change packages without bumping version number?
> 
> If you would care to verify that in each case, removing the restriction fetches
> a valid tarball from a Gentoo mirror, and supply a patch, we can look at making
> this change.
Would you like a all-in-one patch or a bunch of patches each against a particular ebuild?

BTW, the following ebuild should be excluded from the list: 
/usr/portage/x11-libs/ViewKlass/ViewKlass-0.7.0.ebuild

Comment 4 Donnie Berkholz (RETIRED) gentoo-dev 2007-01-22 06:04:07 UTC
(In reply to comment #3)
> Would you like a all-in-one patch or a bunch of patches each against a
> particular ebuild?

Thanks for asking. One patch per category, or one for the whole tree, would be most useful.
Comment 5 Zhang Le (RETIRED) gentoo-dev 2007-01-22 17:05:00 UTC
Created attachment 107807 [details, diff]
remove RESTRICT="mirror" from Xorg packages' ebuilds

This is an all-in-one patch. Please double check it before apply.
Thanks!
Comment 6 Donnie Berkholz (RETIRED) gentoo-dev 2007-01-22 18:32:38 UTC
(In reply to comment #5)
> Created an attachment (id=107807) [edit]
> remove RESTRICT="mirror" from Xorg packages' ebuilds
> 
> This is an all-in-one patch. Please double check it before apply.
> Thanks!

Just want to make sure -- you did check that all tarballs on Gentoo mirrors are valid?
Comment 7 Zhang Le (RETIRED) gentoo-dev 2007-01-23 01:16:51 UTC
(In reply to comment #6)
> Just want to make sure -- you did check that all tarballs on Gentoo mirrors are
> valid?
Please refer to http://www.gentoo.org/proj/en/infrastructure/mirrors/overview-distfile.xml.
If you're ebuild contains restrict="mirror" the file will not be mirrored.
So only after applying this patch, these packages will be mirrored on gentoo mirrors.
Comment 8 Donnie Berkholz (RETIRED) gentoo-dev 2007-01-23 06:07:24 UTC
(In reply to comment #7)
> Please refer to
> http://www.gentoo.org/proj/en/infrastructure/mirrors/overview-distfile.xml.
> If you're ebuild contains restrict="mirror" the file will not be mirrored.
> So only after applying this patch, these packages will be mirrored on gentoo
> mirrors.

Perhaps you did not clearly understand the reasoning for adding the restriction in the first place. It was because a different tarball was on Gentoo mirrors than on xorg.freedesktop.org. If that is still the case, then removing the restriction will break all of these ebuilds. Each ebuild requires manual verification. If that isn't done, the rest of this work is pointless.
Comment 9 Zhang Le (RETIRED) gentoo-dev 2007-01-23 07:06:08 UTC
(In reply to comment #8)
> It was because a different tarball was on Gentoo mirrors
> than on xorg.freedesktop.org. If that is still the case, then removing the
> restriction will break all of these ebuilds. Each ebuild requires manual
> verification. If that isn't done, the rest of this work is pointless.
But Gentoo mirrors don't contain any of these packages now.
Moreover, after removing the restriction, emerge will eventually find packages on freedesktop.org, user still could get the package.
So I am afraid there will be no breakage at all.

Comment 10 Zhang Le (RETIRED) gentoo-dev 2007-01-23 07:20:35 UTC
PS: blame me for being dense if it's a misunderstanding
Comment 11 Donnie Berkholz (RETIRED) gentoo-dev 2007-01-23 23:07:15 UTC
(In reply to comment #9)
> But Gentoo mirrors don't contain any of these packages now.

OK. You have not said before that you confirmed mirrors do not have any of them -- you just quoted some mirroring guide at me.
Comment 12 Zhang Le (RETIRED) gentoo-dev 2007-01-24 03:25:55 UTC
(In reply to comment #11)
> OK. You have not said before that you confirmed mirrors do not have any of them
> -- you just quoted some mirroring guide at me.
Sorry, my fault.
Anyway, glad to see this patch could be merged. 

Comment 13 Raúl Porcel (RETIRED) gentoo-dev 2007-01-24 10:36:18 UTC
Shouldn't be more easy if we use xorg in thirdpartymirrors?
Comment 14 Zhang Le (RETIRED) gentoo-dev 2007-01-29 08:45:07 UTC
(In reply to comment #13)
> Shouldn't be more easy if we use xorg in thirdpartymirrors?
> 
In that case, user have to do some extra configuration in order to use the nearly xorg mirror, i.e. modify /etc/portage/mirrors.
What's worse, maybe there is no xorg mirror nearby, or they don't know where to find nearby xorg mirrors.
All this would bring more trouble to users.

So, IMHO, removing RESTRICT="mirror" from Xorg packages' ebuilds may be a better solution. 

I will be very appreciated if if this patch could be merged. Coz it's extremely  slow to download packages directly from freedesktop.org in Mainland China.
Thanks in advance!
Comment 15 Druggo Yang 2007-02-03 10:29:57 UTC
oh, I really need this patch, pls apply it !
Comment 16 He Jian 2007-02-03 10:32:37 UTC
please apply this patch, i have a fast gentoo mirror nearby, but it is very slow to download things from f.d.o
Comment 17 Joshua Baergen (RETIRED) gentoo-dev 2007-02-04 18:53:33 UTC
The attached patch has been applied.  Thanks for looking into this :)