[opendmarc-users] Patch Comparison and Personal RFC

Juri Haberland juri at sapienti-sat.org
Mon Dec 12 23:50:19 PST 2016


Good morning everyone.

On 2016-12-13 06:42, Scott Kitterman wrote:
> On Monday, December 12, 2016 07:56:57 PM Steve Jenkins wrote:
>> Scott, Juri, and anyone else who feels like chiming in (I'm looking at 
>> you,
>> Andreas):
>> 
>> I've built this simple spreadsheet showing the patches that each of 
>> you
>> suggests including in an opendmarc build that you consider "worthy" of
>> pushing to stable repos , even though it's built using beta source:
>> 
>> https://docs.google.com/spreadsheets/d/1bhlYp_A6j_NF6I9cEAoHsYJV06ooEjkDlsMo
>> 9v67mtQ/edit?usp=sharing
>> 
>> As you can see, for the most part, both agree on the majority of the
>> patches.
>> 
>> But before I make a final decision regarding which patches I believe 
>> should
>> appear in a Fedora/RHEL build, I'd like to hear comments about:
>> 
>> - why you chose to include a patch that the other one didn't, and
>> - why you chose not to include a patch that the other one did
> 
> So here's the differences:
> 
> https://sourceforge.net/p/opendmarc/tickets/159/
> 
> I didn't include it because I didn't have the time to study it.

In hindsight this patch is wrong - it uses an internal variable of the 
autoconf/automake stuff, which it shouldn't. I need to dig deeper into 
this matter to understand how autoconf/automake works. I guess there is 
a problem in the way it is used in OpenDMARC and that is also why 
Scott's approach to use %{prefix} in the ./configure command line didn't 
work as expected.
If you didn't already applied it - leave it out.

btw. the same goes for https://sourceforge.net/p/opendmarc/tickets/179/ 
which is on top of #159.

> https://sourceforge.net/p/opendmarc/tickets/168/
> 
> I included this one, but in retrospect, probably shouldn't have.  It's 
> a new
> feature and I generally don't think we should be distro patching those 
> in.  I
> don't plan to drop it, since I've shipped it, but I don't particularly
> recommend you add it.
> 
> Not sure what I was thinking at the time.

He he, I wondered why you included that one, too, but didn't want to 
start a discussion, because I was glad that there was some progress at 
least at the distro/packaging end...

> https://sourceforge.net/p/opendmarc/tickets/171/
> 
> Seemed low priority, but probably worthwhile including. The only 
> concern is if
> later msk decides to change the code to match the docs rather than 
> change the
> docs to match the code.

It is low priority, but annoying if you are try to debug some mail flow 
and run into this problem. And yes, I understand your concerns, but 
maybe just create precedents?

> https://sourceforge.net/p/opendmarc/tickets/193/
> 
> I think I missed this one by accident.

The patch is fairly new, but we had a couple of users on the ML 
complaining about scripts failing with MySQL running with strict mode 
enabled.

> https://sourceforge.net/p/opendmarc/tickets/197/
> 
> This one I left alone since there's no telling which of the two msk 
> will keep.
> I thought it better to have two than to remove one and have it change 
> later.

Fair enough - or just create another precedent ;)


   Juri


More information about the opendmarc-users mailing list