Skip to content

Commit

Permalink
remove 'hdrcopy' from autotools build system (#428)
Browse files Browse the repository at this point in the history
  • Loading branch information
@farindk
farindk committed Nov 20, 2023
1 parent a5bfcad commit 8a6176e
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions dec265/Makefile.am
View file
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@

bin_PROGRAMS = dec265 hdrcopy
bin_PROGRAMS = dec265 # hdrcopy

AM_CPPFLAGS = -I$(top_srcdir)/libde265 -I$(top_srcdir)

Expand All @@ -9,11 +9,11 @@ dec265_LDFLAGS =
dec265_LDADD = ../libde265/libde265.la -lstdc++
dec265_SOURCES = dec265.cc

hdrcopy_DEPENDENCIES = ../libde265/libde265.la
hdrcopy_CXXFLAGS =
hdrcopy_LDFLAGS =
hdrcopy_LDADD = ../libde265/libde265.la -lstdc++
hdrcopy_SOURCES = hdrcopy.cc
# hdrcopy_DEPENDENCIES = ../libde265/libde265.la
# hdrcopy_CXXFLAGS =
# hdrcopy_LDFLAGS =
# hdrcopy_LDADD = ../libde265/libde265.la -lstdc++
# hdrcopy_SOURCES = hdrcopy.cc

if HAVE_VIDEOGFX
dec265_CXXFLAGS += $(VIDEOGFX_CFLAGS)
Expand Down

2 comments on commit 8a6176e

@fancycode
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will remove hdrcopy.c from the source tarball.

Better would have been to move it to noinst_PROGRAMS to still include and build it, but not install it from make install.

@farindk
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's ok to remove this also from the source tarball since it's probably of no use for anybody. Even myself, I don't really remember what I should use it for.

Please sign in to comment.