You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2011/10/28 03:42:19 UTC

[Bug 6685] New: [review] Sync Debian packaging from v3.3.2-2 to trunk

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6685

             Bug #: 6685
           Summary: [review] Sync Debian packaging from v3.3.2-2 to trunk
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Packaging: debian
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: Darxus@ChaosReigns.com
    Classification: Unclassified


Created attachment 4989
  --> https://issues.apache.org/SpamAssassin/attachment.cgi?id=4989
patch

>From the Debian changelog:

spamassassin (3.3.2-2) unstable; urgency=low

  * Remove dependencies on libdigest-sha1-perl, since it's being 
    removed from Debian. (Closes: #629612)

 -- Noah Meyerhans <no...@debian.org>  Thu, 18 Aug 2011 17:07:05 -0700


Very minor.  Done the same as bug 6647 and bug 6593.  Thoroughly documented at
http://wiki.apache.org/spamassassin/SyncDebianPackaging

Not tested, just as before, because of minimal possibility of breakage and
significantly increased ease of testing after commit.

All modified files:
+++ debian/control
+++ debian/README.source
+++ debian/changelog

Current delta from Debian is the same as last time, removing two patches
because they have already been applied to trunk:  85_disable_SSLv2 and
50_sa-learn_fix_empty_list_handling.  Exact same packaging is used by Ubuntu.

Shouldn't need any votes, just a commit, since I don't have the access.

-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6685] [review] Sync Debian packaging from v3.3.2-2 to trunk

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6685

--- Comment #4 from Darxus <Da...@ChaosReigns.com> 2011-10-28 15:03:59 UTC ---
Yes, but this patch was created by a very simple process of deleting the
existing /debian/ directory in trunk, copying over the contents of the
spamassassin_*.debian.tar.gz from Debian, and doing a diff (after deleting the
two no-longer necessary patches).  I don't see how it's worth deleting that
comment every time from now on.  It's just extra work maintaining a delta
between the latest packaging info from Debian, and a slightly older version of
the same thing from the same source, that affects nothing.

(Synced update to launchpad, queued build.)

-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6685] [review] Sync Debian packaging from v3.3.2-2 to trunk

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6685

Darxus <Da...@ChaosReigns.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |Darxus@ChaosReigns.com

--- Comment #2 from Darxus <Da...@ChaosReigns.com> 2011-10-28 14:32:36 UTC ---
(In reply to comment #1)
> I don't see how the README.source is relevant to the SA project.  

It's not.  I just don't think it's worth the additional work to maintain the
difference from the debian packaging.  But no objection to you leaving it out. 
Just seems silly.

> Otherwise committed:

Thanks, rebuilding.

-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6685] [review] Sync Debian packaging from v3.3.2-2 to trunk

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6685

--- Comment #3 from Kevin A. McGrail <km...@pccc.com> 2011-10-28 14:45:11 UTC ---
(In reply to comment #2)
> (In reply to comment #1)
> > I don't see how the README.source is relevant to the SA project.  
> 
> It's not.  I just don't think it's worth the additional work to maintain the
> difference from the debian packaging.  But no objection to you leaving it out. 
> Just seems silly.

It's a note from the DEBIAN packagers explaining their differences downstream. 
To me it seems silly to include it upstream.

-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6685] [review] Sync Debian packaging from v3.3.2-2 to trunk

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6685

Kevin A. McGrail <km...@pccc.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |kmcgrail@pccc.com
         Resolution|                            |FIXED

--- Comment #1 from Kevin A. McGrail <km...@pccc.com> 2011-10-28 14:28:10 UTC ---
I don't see how the README.source is relevant to the SA project.  

Otherwise committed:

svn commit -m 'Debian build file changes per bug 6685' 
Sending        debian/changelog
Sending        debian/control
Transmitting file data ..
Committed revision 1190353.

-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6685] [review] Sync Debian packaging from v3.3.2-2 to trunk

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6685

--- Comment #5 from Kevin A. McGrail <km...@pccc.com> 2011-10-28 15:17:56 UTC ---
(In reply to comment #4)
> Yes, but this patch was created by a very simple process of deleting the
> existing /debian/ directory in trunk, copying over the contents of the
> spamassassin_*.debian.tar.gz from Debian, and doing a diff (after deleting the
> two no-longer necessary patches).  I don't see how it's worth deleting that
> comment every time from now on.  It's just extra work maintaining a delta
> between the latest packaging info from Debian, and a slightly older version of
> the same thing from the same source, that affects nothing.
> 
> (Synced update to launchpad, queued build.)

We'll agree to disagree ;-)

-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.