I don't think it's a good idea.
./autogen.sh is run by everyone any time you want to build the
binary.
But we want a target to be run only when really needed.
If an autotools solution can't be found, you can use a dedicated
script
(release.sh) just to save the commands to be run before a new
release.
On Mon, Jan 24 2022 at 10:21:37 -0600, Jeremiah Benham
<jeremiahbenham@gmail.com> wrote:
> Can we place the commands in autogen.sh?
>
> Jeremiah
>
> On Thu, Jan 20, 2022, 9:58 AM Richard Shann
> <richard@rshann.plus.com>
> wrote:
> > On Thu, 2022-01-20 at 11:38 +0100, Federico Bruni wrote:
> > >
> > >
> > > Il giorno gio 20 gen 2022 alle 09:33:14 +0000, Richard Shann
> > > <richard@rshann.plus.com> ha scritto:
> > > > On Wed, 2022-01-19 at 11:18 -0600, Jeremiah Benham wrote:
> > > > > Richard,
> > > > > I am not seeing his comment in the git logs.
> > > >
> > > > Sorry, this is my mistake - there is some git patch command
> > I
> > > > should
> > > > have used to transfer the comment; the comment is this:
> > > >
> > > > "Something like this should be integrated in the
> > Makefile.am:
> > > >
> > > > news-to-metainfo-releases:
> > > > @cd $(builddir) &&
> > > > appstreamcli news-to-metainfo --limit=6
> > $(srcdir)/NEWS
> > \\
> > > > org.denemo.Denemo.appdata.xml
> > org.denemo.Denemo.appdata.xml
> > > > "
> > > >
> > > > > I am a little confused
> > > > > about what needs to be done in autotools. What needs to
> > happen?
> > > >
> > > > I'm not up to speed on this desktop configuration stuff -
> > I'll
> > send
> > > > this to Federico and he'll be able to explain.
> > > >
> > > > Richard
> > > >
> > > >
> > > >
> > > > > He
> > > > > said he wants to make a target. Does he want it to be
> > installed
> > > > > or
> > > > > is
> > > > > there some processing that needs to happen before
> > something
> > gets
> > > > > installed. Is it processed before build time or during
> > install?
> > > > > I am
> > > > > a
> > > > > little confused. Is it this:
> > > > > @cd $(builddir) &&
> > > > > appstreamcli news-to-metainfo --limit=6
> > $(srcdir)/NEWS \\
> > > > > org.denemo.Denemo.appdata.xml
> > org.denemo.Denemo.appdata.xml
> > > > >
> > > > > Thanks,
> > > > > Jeremiah
> > > > >
> > >
> > >
> > > Hi Jeremiah
> > >
> > > Right now org.denemo.Denemo.appdata.xml.in is processed once
> > by
> > > intltool, which takes the translated strings from the PO
files
> > and
> > > add
> > > them to the final .xml file in the builddir (which will be
> > > installed).
> > >
> > > I want to add one more processing during build time, in order
> > to
> > add
> > > the release notes to the final .xml file.
> >
> > If all this is just to copy something from the file NEWS to the
> > file
> > org.denemo.Denemo.appdata.xml.in then I could just do this while
> > copying from ChangeLog to NEWS (I put the release notes in
> > ChangeLog
> > because Debian was requiring a change to ChangeLog from one
> > release
> > to
> > the next and I added them to NEWS because I thought you needed
> > them
> > there for flatpak. I can just as easily update
> > org.denemo.Denemo.appdata.xml.in
> > if that makes life easier). What <field></field>s would need be
> > needed?
> > - I'm assuming once the <field>s are set up it is just a matter
> > of
> > copying text (well, escaping <>& or avoiding them).
> >
> > Richard
> >
> > > My first approach - what I thought it might be easier - was
> > simply
> > > overriding the .xml file processed by intlttol using the
> > appstreamcli
> > > news-to-metainfo command above. As you can see, the file
> > > $(builddir)/org.denemo.Denemo.appdata.xml is both the input
> > and
> > the
> > > output file.
> > > But that command is not run, because I don't understand how
> > Autotools
> > > targets work.
> > >
> > > Thanks
> > > Federico
> > >
> > >
> > >
> > >
> > >
> >
> >