FS#15984 - Regression: makepkg --source still fetches build files from the web

Attached to Project: Pacman
Opened by Devin Cofer (Ranguvar) - Wednesday, 19 August 2009, 03:29 GMT
Last edited by Allan McRae (Allan) - Saturday, 22 May 2010, 12:49 GMT
Task Type Bug Report
Category makepkg
Status Closed
Assigned To Xavier (shining)
Dan McGee (toofishes)
Allan McRae (Allan)
Architecture All
Severity Medium
Priority Normal
Reported Version 3.3.0
Due in Version 3.4.0
Due Date Undecided
Percent Complete 100%
Votes 3
Private No

Details

Summary and Info:
From 'man makepkg':
--source
Do not actually build the package, but build a source-only tarball that does not include sources that can
be fetched via a download URL. This is useful for passing a single tarball to another program such as a
chroot, remote builder, or a tarball upload.
So, makepkg --source should not pull in build files from the web. However, since pacman-3.3.0 I believe, it does. I don't believe makepkg --source places those files in the source archive it creates, but it's still annoying when it downloads the source files for a package before it will let you use --source to make an AUR package.


Steps to Reproduce:
Pick any AUR package that has a web file in its source=(), download the PKGBUILD and such from the AUR, and makepkg --source.
This task depends upon

Closed by  Allan McRae (Allan)
Saturday, 22 May 2010, 12:49 GMT
Reason for closing:  Fixed
Additional comments about closing:  git commit http://projects.archlinux.org/pacman.git /commit/?id=c1fc0050
Comment by Xavier (shining) - Wednesday, 19 August 2009, 06:21 GMT
see http://projects.archlinux.org/?p=pacman.git;a=commitdiff;h=02acf65ef30048ba3c732628d30215467245e779
"It wouldn't be very nice to ship a PKGBUILD with the wrong checksums."

Or would it?
Comment by Devin Cofer (Ranguvar) - Monday, 07 September 2009, 03:25 GMT
I see the reasoning. However, if it is decided that makepkg --source _should_ fetch all build files,

1.) This should be documented in the man page. Right now it says that external build files will NOT be fetched.
2.) An override switch should exist. I sometimes use mirrors with limited bandwidth, and I already _know_ the sums will check out, or I may be using a metered connection, etc.
Comment by Allan McRae (Allan) - Monday, 07 September 2009, 04:08 GMT
Dan, Xavier: any opinions here?

I am undecided. I use "makepkg --source" only after actually building the package. In that case, the source and md5sums need to have been present anyway. But if it was a minor version update and md5sums were given on a projects homepage and I had limited or costly bandwidth, I might be tempted to skip the building...
Comment by Xavier (shining) - Monday, 07 September 2009, 08:27 GMT
I don't use 'makepkg --source' a lot, but when I do, it is also always after building the package

No matter how simple the update is, there is always something that can go wrong. Too many times, I saw very simple changes, untested, which were actually wrong. This is also the reason why I was against an on-line AUR editor.

That said, I don't care so much. We can keep it as is, or revert the above commit, or add an override, it would not change anything to me.
Ranguvar : what would that override be ?
Comment by Allan McRae (Allan) - Monday, 07 September 2009, 11:10 GMT
Do we just want to use --skipinteg ( FS#15830 ) here too?
Comment by Xavier (shining) - Monday, 07 September 2009, 12:19 GMT
hm why not. sounds alright.
Comment by Gavin Bisesi (Daenyth) - Monday, 07 September 2009, 14:52 GMT
If using --skipinteg to bypass downloading remote files is available, that's fine. Personally I think that should be the default, but I'm fine with a flag
Comment by Dan McGee (toofishes) - Monday, 07 September 2009, 15:01 GMT
We probably should have updated the documentation at the same time we brought in the original "regresssion" patch. Loui wrote it (who gets some credibility for being the AUR maintainer and seeing too many of these), Allan signed off on it, and I merged it, so I think it has a fair amount of support.

Can we at the very least get a documentation update for 3.3.1? And I'm not convinced we should ever allow skipping, but I do see the valid reasons. I just don't want that to become the norm...
Comment by Dan McGee (toofishes) - Monday, 07 September 2009, 15:03 GMT
*\--source*::
Do not actually build the package, but build a source-only tarball that
does not include sources that can be fetched via a download URL. This is
useful for passing a single tarball to another program such as a chroot,
remote builder, or a tarball upload. Because integrity checks are verified,
all source files of the package need to be present or downloadable.
Comment by Xavier (shining) - Monday, 07 September 2009, 15:24 GMT
Should I make a patch with this doc update ?
Comment by Dan McGee (toofishes) - Monday, 07 September 2009, 15:52 GMT
If you and Allan are fine with the wording, then sure. Of course I can make it too; I just wanted to get a +1 or -1 here.
Comment by Gavin Bisesi (Daenyth) - Monday, 07 September 2009, 17:06 GMT
Sounds good to me, but maybe document that --skipinteg will prevent sources being downloaded?
Comment by Xavier (shining) - Monday, 07 September 2009, 17:20 GMT
Well this is not the case currently. Allan only suggested that this --skipinteg option could be extended to cover this case.

And Dan said : "I'm not convinced we should ever allow skipping, but I do see the valid reasons"
So it's not sure at all it will be done.

But in any cases, the doc had to be updated.
Comment by Gavin Bisesi (Daenyth) - Monday, 07 September 2009, 17:29 GMT
Ah, I misread. Fair enough.
Comment by Xavier (shining) - Monday, 07 September 2009, 17:37 GMT
doc patch in my maint-working branch
Comment by Allan McRae (Allan) - Monday, 07 September 2009, 20:28 GMT
ack.
Comment by Allan McRae (Allan) - Saturday, 19 September 2009, 11:50 GMT
Docs will be updated with 3.3.1 (commit 814cd7ae).

So is there a decision of whether we can use --skipinteg here too. I'm +/- 1. Maybe more +1 than -1 as we already have the option there and it makes sense to use it again.
Comment by Gavin Bisesi (Daenyth) - Saturday, 19 September 2009, 12:26 GMT
I'd prefer using --skipinteg, but I think it's probably up to Dan and whoever writes the final patch
Comment by Xavier (shining) - Saturday, 19 September 2009, 12:29 GMT
I think using skipinteg is fine.
The question is rather : do we implement this or not ?
See what Dan said : "And I'm not convinced we should ever allow skipping, but I do see the valid reasons. I just don't want that to become the norm..."

But if no one provides a patch, I guess there is no question :)
Comment by Allan McRae (Allan) - Sunday, 11 October 2009, 11:23 GMT Comment by Dan McGee (toofishes) - Thursday, 17 June 2010, 05:07 GMT
Congrats, you just wasted 6 people's time with your "Don't reopen" in a *reopen request*. Please never do that again. And read why it was closed in the first place.

Loading...