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

make dist command will now add docs folder and README.md file in the output zip #846

Closed
wants to merge 1 commit into from

Conversation

sukhdeepg
Copy link
Contributor

I've edited the Makefile.am to add docs folder and README.md file in zip that is created after make dist command.

@quozl
Copy link
Contributor

quozl commented Jul 18, 2019

Thanks. Tested. Changes to .tar.xz file were;

--- old 2019-07-18 15:23:52.485607116 +1000
+++ new 2019-07-18 15:23:58.245546253 +1000
@@ -49,6 +48,24 @@
 sugar-0.114/data/sugar-72.gtkrc
 sugar-0.114/data/sugar.desktop
 sugar-0.114/data/sugar.xml.in
+sugar-0.114/docs/
+sugar-0.114/docs/controls.txt
+sugar-0.114/docs/debian.md
+sugar-0.114/docs/debian-packaging-example.md
+sugar-0.114/docs/debian-packaging-example.md.~1~
+sugar-0.114/docs/debian-packaging-example.md.~2~
+sugar-0.114/docs/design.txt
+sugar-0.114/docs/development-environment.md
+sugar-0.114/docs/fedora.md
+sugar-0.114/docs/GPL-C.txt
+sugar-0.114/docs/GPL-python.txt
+sugar-0.114/docs/LGPL-C.txt
+sugar-0.114/docs/LGPL-python.txt
+sugar-0.114/docs/README.md
+sugar-0.114/docs/release_howto.txt
+sugar-0.114/docs/rpi.md
+sugar-0.114/docs/rpi-soas.md
+sugar-0.114/docs/ubuntu.md
 sugar-0.114/extensions/
 sugar-0.114/extensions/cpsection/
 sugar-0.114/extensions/cpsection/aboutcomputer/
@@ -180,6 +197,7 @@
 sugar-0.114/Makefile.am
 sugar-0.114/Makefile.in
 sugar-0.114/missing
+sugar-0.114/NEWS
 sugar-0.114/po/
 sugar-0.114/po/ach.po
 sugar-0.114/po/adh.po
@@ -358,6 +376,7 @@
 sugar-0.114/po/zh_TW.po
 sugar-0.114/po/zu.po
 sugar-0.114/py-compile
+sugar-0.114/README.md
 sugar-0.114/src/
 sugar-0.114/src/jarabe/
 sugar-0.114/src/jarabe/apisocket.py

Inclusion of editor backup files is a minor irritation that I'll have to live with.

Merged by hand with rewrite of commit message, see ba66588.

In future for automation, please add "fixes #821" to pull request, or "Fixes #821" to commit message. Please also see making commits for how to write commit messages.

@quozl quozl closed this Jul 18, 2019
@sukhdeepg
Copy link
Contributor Author

Understood. Thanks!

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