Skip to content

Publish.el #55

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Publish.el #55

wants to merge 2 commits into from

Conversation

Boruch-Baum
Copy link
Contributor

Hi @tuhdo,

I think this commit faithfully reproduces the html from org; it might even end up being identical to the one you originally used.

HOWEVER:

  1. The files produced ARE different from the ones currently staged. I attribute this to changes made directly to the HTML. If that's true, then the org files need to be updated accordingly.

  2. This exercise made evident that not all of the org files in the project were staged, and not all HTML files have org files staged. Specifically:

2.1) HTML files missing but ORGg files present: cedet.html, c-ide-irony-back.html, ebrowse.html, theindex.html

2.2) ORG files missing but HTML files present: archive.html

* Sync org file file html file.

* Remove text that misled me to think that no further customization of
  the demo package was necessary in order for it to function as
  described in the guide.

* Correct dependency ambiguity about helm and helm-gtags.

* Move a few paragraphs from section 'Source code navigation' to
  section 'Quick setup'.

* Describe what tags are BEFORE beginning to use them.

* Edits to section describing tag creation
* Create a `org-publish-project-alist' to enable publishing the
  project in a manner consistent with the current output.
@tuhdo
Copy link
Owner

tuhdo commented Feb 14, 2017

@Boruch-Baum Sorry for the delay. I was having a baby in December and was focusing on finishing this book: https://tuhdo.github.io/os01/. It might interest you.

I will try to review your PR asap, after the book is settled.

@Boruch-Baum
Copy link
Contributor Author

Congratulations on the baby!

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

Successfully merging this pull request may close these issues.

2 participants