mbox series

[0/3] tools/ocaml: Incremental build fixes

Message ID 20240823105157.991275-1-andrew.cooper3@citrix.com (mailing list archive)
Headers show
Series tools/ocaml: Incremental build fixes | expand

Message

Andrew Cooper Aug. 23, 2024, 10:51 a.m. UTC 
Various extra build bugfixes found while reviewing the `Stabilize Oxenstored's
interface with` series.

Andrew Cooper (3):
  tools/ocaml: Drop OCAMLOPTFLAG_G invocation
  tools/ocaml: Drop o= rune
  tools/ocaml: Fix the version embedded in META files

 tools/ocaml/Makefile.rules | 5 +++--
 tools/ocaml/common.make    | 7 ++-----
 2 files changed, 5 insertions(+), 7 deletions(-)


base-commit: 5231765a4e6528a3208e49885b9eeff42519a6c1

Comments

Christian Lindig Aug. 23, 2024, 10:53 a.m. UTC | #1 
> On 23 Aug 2024, at 11:51, Andrew Cooper <andrew.cooper3@citrix.com> wrote:
> 
> Various extra build bugfixes found while reviewing the `Stabilize Oxenstored's
> interface with` series.
> 
> Andrew Cooper (3):
>  tools/ocaml: Drop OCAMLOPTFLAG_G invocation
>  tools/ocaml: Drop o= rune
>  tools/ocaml: Fix the version embedded in META files
> 
> tools/ocaml/Makefile.rules | 5 +++--
> tools/ocaml/common.make    | 7 ++-----
> 2 files changed, 5 insertions(+), 7 deletions(-)
> 
> 
> base-commit: 5231765a4e6528a3208e49885b9eeff42519a6c1
> -- 
> 2.39.2
> 

Acked-by: Christian Lindig <christian.lindig@cloud.com>
Edwin Torok Aug. 23, 2024, 1:58 p.m. UTC | #2 
The Makefile changes look fine to me as well.

Reviewed-by: Edwin Török <edwin.torok@cloud.com>

On Fri, Aug 23, 2024 at 11:53 AM Christian Lindig
<christian.lindig@cloud.com> wrote:
>
>
>
> > On 23 Aug 2024, at 11:51, Andrew Cooper <andrew.cooper3@citrix.com> wrote:
> >
> > Various extra build bugfixes found while reviewing the `Stabilize Oxenstored's
> > interface with` series.
> >
> > Andrew Cooper (3):
> >  tools/ocaml: Drop OCAMLOPTFLAG_G invocation
> >  tools/ocaml: Drop o= rune
> >  tools/ocaml: Fix the version embedded in META files
> >
> > tools/ocaml/Makefile.rules | 5 +++--
> > tools/ocaml/common.make    | 7 ++-----
> > 2 files changed, 5 insertions(+), 7 deletions(-)
> >
> >
> > base-commit: 5231765a4e6528a3208e49885b9eeff42519a6c1
> > --
> > 2.39.2
> >
>
> Acked-by: Christian Lindig <christian.lindig@cloud.com>
>