Re: getting ready for Denemo 2.6 flatpak

[Richard Shann]

I was wondering what the situation is now with flatpak - when I tried
just now I got release 2.5.0 which seems to be using libevince instead
of libatril (the symptom being that the lilypond links in the Print
View do not work).
Richard

On Mon, 2022-01-24 at 22:59 +0100, Federico Bruni wrote:
> 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
> > >  >
> > >  >
> > >  >
> > >  >
> > >  >
> > > 
> > > 
> 
>