Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#11 has some problem. #12

Open
yeseuleee opened this issue Jun 19, 2019 · 8 comments
Open

#11 has some problem. #12

yeseuleee opened this issue Jun 19, 2019 · 8 comments

Comments

@yeseuleee
Copy link

yeseuleee commented Jun 19, 2019

I have installed ach (version before commit 592fe29 ).
When I was install it, I don't experienced any problem.
But when I have installed ach today, I experienced some problem.

The error is "2555: *** missing separator. Stop."
So I was edit that line in ach/Makefile.am and in ach/doc/Makefile.am (to version before commit 592fe29.)
And I was success to 'make' instruction.
Than would you please fix this problem in version 592fe29

@ndantam
Copy link
Member

ndantam commented Jun 19, 2019

Could you please give some additional detail to help reproduce the problem:

  • Does the problem occur on a clean clone?
  • What OS / distribution are you using?
  • Which build step is failing (autoreconf, configure, make)?

@yeseuleee
Copy link
Author

Sorry. I was add more detail.
Does the problem occur on a clean clone?
-> Yes. The problem occur on a clean clone that recently version.
What OS / distribution are you using?
-> My OS is Ubuntu 16.04 LTS
Which build step is failing (autoreconf, configure, make)?
-> Failing in "make" build step

Really Thankyou!

@ndantam
Copy link
Member

ndantam commented Jun 20, 2019

Thanks for the details. I think the best approach is to set up dockerized integration tests (as we've done for OMPL, TMKit, and Amino) to reproduce this issue and catch similar problems in the future. Unfortunately, I'm traveling the for the next week and have some tight deadlines soon, so I'm not sure if I can get to this right way. But, it sounds like you are OK for now using the previous commit or the last release version?

@yeseuleee
Copy link
Author

yeseuleee commented Jun 20, 2019

Yes. The problem wasn't occur in previous commit. So I use previous revision. Just I wanted let you know.
Have a nice trip!

@acxz
Copy link
Contributor

acxz commented Jun 20, 2019

@yeseuleee those commits were needed due to upgrades to autoconf and man2html.
Can you list the versions you are currently using for these packages? i.e. what is the output of autoconf --version and the last section of the output of man2html --help? Maybe the fix might just be a simple sudo apt update && sudo apt install autoconf man2html on Ubuntu 16.04.

EDIT: I am running autoconf v2.69 and man2html v3.0.1 and things work fine on my machine (ArchLinux). I checked the ubuntu 16.04 packages and they also use autoconf v2.69 but their man2html is v1.6g8.
The autoreconf step is working for you, but the make is failing, hmm.

Do you mind posting the entire output of make? Maybe in a gist or something?

@briansoe66
Copy link

I receive the same error. This is the only output from running make:
Makefile:2525: *** missing separator. Stop.

Line 2525:
@DX_RULES@

Checking out commit 4a38f91 compiles fine.

System:
Ubuntu 18.04
automake 1.15.1
autoconf 2.69
man2html 1.6g-11)

@crackwitz
Copy link
Contributor

I've seen that error too, but only on my advisor's computer, which is an old ubuntu.

it seems to be related to doxygen. see if installing doxygen makes a difference.

@briansoe66
Copy link

I have doxygen 1.8.13 installed. I think the issue is related to autoconf-archive. I do not have autoconf-archive installed because it broke a different part of the build process (sorry, I dont recall what).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants