On Tuesday 07 April 2009 04:28:42 Patrick Schoenfeld wrote: > Hi, > > Daniel Leidert wrote: > > Normally people add the get-orig-source in debian/rules, because the > > original tarball needs to be repackaged. In such cases, trying to get > > the upstream tarball from the location in debian/watch is probably > > not, what the maintainer wants, because then he gets the original > > tarball, that doesn't contain the necessary change. > > > > What is your opinion? Are there limitations that need to be kept in mind > > when trying to provide a patch? > > I agree that this is an interesting feature which is worth beeing > added. The only question is weither we should do this by default or not. > Lets say we've added a heuristic that tries to detect weither a makefile > has a get-orig-source target (e.g. by issueing something like make -n > debian/rules get-orig-source and looking at the exit code). So far so > good. Technically no big problem. > > I just fear that not everybody uses the get-orig-source target in the > same way. For example I know that some people have the perception > that get-orig-source should download a specific version, while others > make it download the newest source. In the first case this would break > the usual uscan behaviour, because uscan wouldn't download the version > it reports but another one that is hardcoded in the rules file. > So possibly this should be made a command line paramater, or a > configuration option, or both. > > What do you think? > > Andres Meija wrote: > > # Run a script (such as debian/rules) to download the orig tarball > > use-download-script=debian/rules \ > > use-download-script-options="get-orig-source" > > I'm unsure weither this makes sense or not. > > On the one side I have the fear that uscan doesn't work properly > if maintainers use get-orig-source different. But on the other side > I think its better to let the one who invocates uscan decide weither he > wants to take the risk to get the wrong source. > Additional I don't see much benefit in making it configurable. We have > debian/rules and the get-orig-source target defined in our policy > so we should stick with it.
Seems the get-orig-source target is not as well defined as a lot of people are unsure what it should do exactly (see bug #466550). My suggestion was to support something like, use-check-script=debian/<some_script> args ... use-download-script=debian/<some_script> args ... Supporting this for get-orig-source targets can be like, use-download-script=debian/rules get-orig-source I think one issue raised was that the DEHS servers could possibly not have the proper dependencies to run the many scripts that could be written (whether they were implemented in a get-orig-source target or in some other script). If that is the case, I suggest we implement a way to get these lines ignored, perhaps when the --dehs option is set. -- Regards, Andres
signature.asc
Description: This is a digitally signed message part.