[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-07-30 15:59 EST --- multiget-1.2.0-3.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added Status|CLOSED |CLOSED Resolution|NEXTRELEASE |CURRENTRELEASE Fixed In Version||1.2.0-3.fc8 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-07-30 16:05 EST --- multiget-1.2.0-3.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added Status|ON_QA |CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-07-02 13:05 EST --- multiget-1.2.0-3.fc9 has been submitted as an update for Fedora 9 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-07-02 13:07 EST --- multiget-1.2.0-3.fc8 has been submitted as an update for Fedora 8 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-07-02 13:09 EST --- multiget-1.2.0-3.fc8 has been submitted as an update for Fedora 8 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Additional Comments From [EMAIL PROTECTED] 2008-07-02 23:15 EST --- multiget-1.2.0-3.fc8 has been pushed to the Fedora 8 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update multiget'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F8/FEDORA-2008-6036 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-06-30 12:11 EST --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-29 03:58 EST --- Here is my pre-review of a package. https://bugzilla.redhat.com/show_bug.cgi?id=448717 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-06-29 04:52 EST --- Okay, your pre-review seems good to some extent for initial comments. This package (multiget) is APPROVED by me Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from Get a Fedora Account. At a point a mail should be sent to sponsor members which notifies that you need a sponsor. At the stage, please also write on this bug for confirmation that you requested for sponsorship and your FAS (Fedora Account System) name. Then I will sponsor you. If you want to import this package into Fedora 8/9, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on koji Fedora rebuilding system). If you have questions, please ask me. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2008-06-29 09:39 EST --- (In reply to comment #17) Then I will sponsor you. As written in comment #2 I am willing to sponsor Guido, in fact I already did, so I'm removing FE-NEEDSPINSOR blocker now. Guido and me know each other from real life and we are both German, which simplifies communication a lot. Nevertheless thanks a lot for offering sponsorship to Guido and for helping out with this review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-29 09:57 EST --- (In reply to comment #18) As written in comment #2 I am willing to sponsor Guido, in fact I already did, so I'm removing FE-NEEDSPINSOR blocker now. Guido and me know each other from real life and we are both German, which simplifies communication a lot. Ah, that is much better! Thank you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED] ||oglemail.com) --- Additional Comments From [EMAIL PROTECTED] 2008-06-28 12:20 EST --- ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEEDINFO|ASSIGNED Flag|needinfo?([EMAIL PROTECTED]| |oglemail.com) | --- Additional Comments From [EMAIL PROTECTED] 2008-06-28 14:55 EST --- Sorry for the delay. My review will be posted during the next week. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-19 14:05 EST --- From the reply of the upstream we can admit that this program is licensed under GPLv2. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-13 13:34 EST --- Okay, as it seems that at least this is licensed under GPLv2 (however I really want the reply from the upstream), this package itself is okay. Then: - NOTE: Before being sponsored: This package will be accepted with another few (or no) work. But before I accept this package, someone (I am a candidate) must sponsor you. Once you are sponsored, you have the right to review other submitters' review requests and approve the packages formally. For this reason, the person who want to be sponsored (like you) are required to show that you have an understanding of the process and of the packaging guidelines as is described on : http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored Usually there are two ways to show this. A. submit other review requests with enough quality. B. Do a pre-review of other person's review request (at the time you are not sponsored, you cannot do a formal review) When you have submitted a new review request or have pre-reviewed other person's review request, please write the bug number on this bug report so that I can check your comments or review request. Fedora package collection review requests which are waiting for someone to review can be checked on: http://fedoraproject.org/PackageReviewStatus/NEW.html (NOTE: please don't choose Merge Review) Review guidelines are described mainly on: http://fedoraproject.org/wiki/Packaging/ReviewGuidelines http://fedoraproject.org/wiki/Packaging/Guidelines http://fedoraproject.org/wiki/Packaging/ScriptletSnippets -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-06-12 02:59 EST --- The package on comment 0 does not build. http://koji.fedoraproject.org/koji/taskinfo?taskID=658540 I just tried to rebuild this package, however - This package seems to apply a patch on Makefiles after calling configure, which is not good in general. You should apply a patch against Makefile.ins. Also, $RPM_OPT_FLAGS differs between on i386 system and on x86_64 system. - The line like #%configure does not prevent configure from being executed (actually from build.log you can see that configure is actually called). This is rpm feature. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-12 05:47 EST --- Thanks for your comments, Mamoru. I've changed the spec and the patch. You can find them at Spec URL: http://www.soglatec.de/fedora/multiget.spec SRPM URL: http://www.soglatec.de/fedora/multiget-1.2.0-2.fc9.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2008-06-12 11:13 EST --- Well, from where can we confirm the license of this package? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-12 11:17 EST --- (In reply to comment #5) Well, from where can we confirm the license of this package? Scratch this comment :( -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-12 12:53 EST --- Well, after all the license issue is pending. It seems that this package is licensed under at least GPLv2, however now we distinguish between GPLv2 and GPLv3 and GPLv2+. Would you ask the upstream to clarify under what license this package is released? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-12 13:10 EST --- One of the reason I am asking is that while some files (including src/mainframe.cpp ) says that they are licensed under GPLv2+, src/mainframe.cpp also includes: -- 9 /* 10 * This program is free software; you can redistribute it and/or modify 11 * it under the terms of the GNU General Public License as published by 12 * the Free Software Foundation; either version 2 of the License, or 13 * (at your option) any later version. 1235 info.SetVersion(wxT(v1.2.0)); 1236 info.SetLicence(wxT(GPLv2)); 1237 info.SetWebSite(wxT(http://multiget.sourceforge.net;)); which seems to be saying as GPLv2 (although the header of the file also says GPLv2+) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-12 13:13 EST --- (In reply to comment #8) One of the reason I am asking is that while some files (including src/mainframe.cpp ) says that they are licensed under GPLv2+, src/mainframe.cpp also includes: I see. I've just sent an email and will get back when I have some news. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: multiget - A multi-thread http/ftp file downloader
https://bugzilla.redhat.com/show_bug.cgi?id=450410
--- Additional Comments From [EMAIL PROTECTED] 2008-06-12 13:49 EST ---
Thanks. Then for 1.2.0-2:
Except for license issue:
* Source3
- Please specify the URL of Source3 (or write as a comment
how you received Source3)
* Buildroot
- Please follow
http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot
* BuildRequires
- Currently gtk2-devel seems to pull automake, however
for fixing broken install-sh symlink I think explicitly
adding BuildRequires: automake is preferred.
* Perl module BRs:
- For perl modules BuildRequires, please follow
http://fedoraproject.org/wiki/Packaging/Perl#requiresandprovides
(In short, BuildRequires: perl-XML-Parser must be
BuildRequires: perl(XML::Parser) )
* Patch0 vs %optflags vs %configure
- You seem to add %Patch0 to honor %optflags, however this Patch0
is not needed.
The reason your srpm does not honor %optflags is that %configure
is called at %prep, not at %build. Actually %configure sets
CPPFLAGS, however written at %prep, all those variables are
reset.
Move %configure to %build and then Patch0 is no longer needed.
* Macros
- Use macros. /usr/share must be %{_datadir}.
* Timestamps
- When using cp or install commands, add -p option to keep
timestamps on installed files.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: multiget - A multi-thread http/ftp file downloader
https://bugzilla.redhat.com/show_bug.cgi?id=450410
--- Additional Comments From [EMAIL PROTECTED] 2008-06-12 15:20 EST ---
(In reply to comment #10)
* Source3
- Please specify the URL of Source3 (or write as a comment
how you received Source3)
Wrote a comment in the spec. Actually there isn't an icon provided within the
source of 1.2.0. So I took the one from 1.1.4
* Buildroot
- Please follow
http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot
Done.
* BuildRequires
- Currently gtk2-devel seems to pull automake, however
for fixing broken install-sh symlink I think explicitly
adding BuildRequires: automake is preferred.
Fixed.
* Perl module BRs:
- For perl modules BuildRequires, please follow
http://fedoraproject.org/wiki/Packaging/Perl#requiresandprovides
(In short, BuildRequires: perl-XML-Parser must be
BuildRequires: perl(XML::Parser) )
Fixed. Will be carefully done in future packages.
* Patch0 vs %optflags vs %configure
- You seem to add %Patch0 to honor %optflags, however this Patch0
is not needed.
The reason your srpm does not honor %optflags is that %configure
is called at %prep, not at %build. Actually %configure sets
CPPFLAGS, however written at %prep, all those variables are
reset.
Move %configure to %build and then Patch0 is no longer needed.
Okay. Got it. So this patch is obsolete now.
* Macros
- Use macros. /usr/share must be %{_datadir}.
Done.
* Timestamps
- When using cp or install commands, add -p option to keep
timestamps on installed files.
Fixed.
Find the latest here:
Spec URL: http://www.soglatec.de/fedora/multiget.spec
SRPM URL: http://www.soglatec.de/fedora/multiget-1.2.0-3.fc9.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||177841 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-08 03:42 EST --- Manuel, the package is orphaned and Mamoru Tasaka, asked if I could take the package, agreed. Furthermore there were significant changes to the spec necessary to get 1.2.0 work. Christoph Wickert offered sponsorship to me and this should be the first Package for me to maintain (though there has been no development in recent days). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450410] Review Request: multiget - A multi-thread http/ftp file downloader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: multiget - A multi-thread http/ftp file downloader https://bugzilla.redhat.com/show_bug.cgi?id=450410 --- Additional Comments From [EMAIL PROTECTED] 2008-06-07 21:20 EST --- Guido, once a program has been accepted in Fedora, the review for newer versions is not usually needed. You should get in touch with multiget's Fedora maintainer and ask him to update to the newer version and / or offer co-maintainership. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
