镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Added empty directories and support for Maven users with non-standard local repo location - #3

Closed
eddgrant wants to merge 3 commits into
apache:trunkfrom
eddgrant:trunk
Closed

eddgrant wants to merge 3 commits into
apache:trunkfrom
eddgrant:trunk

Conversation

@eddgrant

Copy link
Copy Markdown

Hi all,

These commits add some empty directories (which were causing the Maven build to fail) which I think were deleted (Git doesn't support empty directories) when the apache svn repo was moved in to GitHub. I have re-created them each containing a single, empty .gitignore file which allows Git to persist them.

Secondly I have changed the way the local Maven repo is looked up, it was previously hard coded to look in the default location, it now takes the value from the user's settings.xml so it can pick up an overridden value if there is one, else will still use the default if not set in settings.xml

Hope this is helpful - a 'mvn clean install' now completes successfully from a fresh GitHub fork without it being necessary to make any modifications which should be encouraging to new roller users.

Cheers,

Edd

…ow maven to build the planet-business module.

Note these are present in the SVN repo so I'm wondering if they were removed when the repo was moved to GitHub as Git doesn't allow empty directories (hence the empty .gitignore file in each directory)
…cating it through the ${settings.localRepository} property.
…ow maven to build the weblogger-business module.

Note these are present in the SVN repo so I'm wondering if they were removed when the repo was moved to GitHub as Git doesn't allow empty directories (hence the empty .gitignore file in each directory)

Also updated main .gitignore to ignore weblogger-web/roller-startup.log which is produced by Maven at build time.
@eddgrant

Copy link
Copy Markdown
Author

Hi,

Just wondering if there's any feedback on this pull request as I notice it hasn't been looked at in over 5 months. Happy to discuss if there's something wrong with the change?

Cheers,

Edd

@nuzayats

Copy link
Copy Markdown
Contributor

Hi Edd, Thanks for the PR and sorry to have kept you waiting for long time.

It looks like Apache Roller team seldom check GitHub. I've filed the one causing the Maven build to fail https://issues.apache.org/jira/browse/ROL-2086 , And I would start working to fix it.

As for making use ${settings.localRepository}, The relevant part has gone in the time of merging Maven projects so it's unnecessary now.

@eddgrant

Copy link
Copy Markdown
Author

Hey,

No worries, thanks for getting back to me and for raising ROL-2086.

Cheers.

Edd

@nuzayats

Copy link
Copy Markdown
Contributor

Hi Edd,

I've just committed the fix of ROL-2086 at commit 38e9c3f . Now you can build Roller using distribution that has been obtained from GitHub.

Thank you again for the contribution,

Kohei

hubot pushed a commit that referenced this pull request Apr 28, 2017
…tpage

Prevent annoying scrolling in EntryEdit page

Thanks!
@snoopdave snoopdave closed this Dec 16, 2018
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.

3 participants