Hi Richard,

Le sam. 16 mai 2020 à 17:34, Richard Sharpe <realrichardsha...@gmail.com> a
écrit :

> On Sat, May 16, 2020 at 6:00 AM João Valverde
> <joao.valve...@tecnico.ulisboa.pt> wrote:
> >
> > Hi Richard,
> >
> > On 15/05/20 23:46, Richard Sharpe wrote:
> > > On Fri, May 15, 2020 at 3:33 PM Peter Wu <pe...@lekensteyn.nl> wrote:
> > >> The "asn1" target rebuilds all asn1 dissectors.
> > >> Alternatively to rebuild a specific one, use a target such as
> "generate_dissector-pkcs1".
> > > Sure, but there seems to be multiple issues.
> > >
> > > 1. The 'documented' command placed in the generated source does not
> > > generate the same source:
> > > 2. make asn1 modifies the source directory, but it seems to me that it
> > > should not do that because that breaks one of the out-of-tree
> > > guarantees that cmake gives you.
> >
> > Normally that would be true for a generated source file, but in this
> > case a choice was made to commit the generated ASN.1 source code to VCS
> > (for efficiency reasons I presume). Therefore the asn1 target is a
> > special one designed only to update the ASN.1 source tree and commit the
> > result.
>
> Shouldn't the developer make that decision?
>

That was a decision taken by the developers years ago, so I'm not sure I
get your point. The idea is to reduce the build time as those dissectors
are not updated that often and generating them takes some time.

Best regards,
Pascal.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe

Reply via email to