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

Add ansible 2.9.7 #1945

Merged
merged 8 commits into from
Jun 23, 2020
Merged

Add ansible 2.9.7 #1945

merged 8 commits into from
Jun 23, 2020

Conversation

alexjorgef
Copy link
Contributor

Add ansible v2.9.7 package

@lazka lazka added the new-package Pull request which adds new package(s) label May 10, 2020
@alexjorgef alexjorgef force-pushed the feature/ansible-pkg branch 2 times, most recently from 456137b to abe2a8d Compare May 22, 2020 08:15
@alexjorgef alexjorgef force-pushed the feature/ansible-pkg branch from 04953a3 to 3c164ea Compare May 25, 2020 01:36
Copy link
Contributor Author

@alexjorgef alexjorgef left a comment

Choose a reason for hiding this comment

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

Exporting MSYS variable will not expose this value to entire msys env? See #1216

ansible/PKGBUILD Outdated

prepare() {
[[ -d ${pkgname}-${pkgver} ]] && rm -rf ${pkgname}-${pkgver}
export MSYS=winsymlinks:lnk
Copy link
Contributor Author

@alexjorgef alexjorgef May 25, 2020

Choose a reason for hiding this comment

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

This work for me, any better solution???

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Alexpux exporting the MSYS variable will not affect the rest of the environment?

Copy link
Member

Choose a reason for hiding this comment

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

@dont do that. Remove this export

Copy link
Contributor Author

@alexjorgef alexjorgef Jun 19, 2020

Choose a reason for hiding this comment

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

After some builds, realize that build step need symlinks as shortcuts:

...
prepare() {
  [[ -d ${pkgname}-${pkgver} ]] && rm -rf ${pkgname}-${pkgver}
  tar zxf "${srcdir}/${pkgname}-${pkgver}.tar.gz" || true
  MSYS=winsymlinks:lnk tar zxf "${srcdir}/${pkgname}-${pkgver}.tar.gz"
}
...

@Alexpux is this a correct solution? more info #1216

@alexjorgef
Copy link
Contributor Author

alexjorgef commented May 25, 2020

This is how i reproduce in my msys shell, any hints why ci build is failing?

(cd ansible; rm -rf pkg/ src/; rm *.pkg.tar.xz; pacman -Rs --noconfirm ansible)
(cd python-wcwidth; rm -rf pkg/ src/; rm *.pkg.tar.xz; pacman -Rs --noconfirm python-wcwidth)
(cd python-jinja; rm -rf pkg/ src/; rm *.pkg.tar.xz; pacman -Rs --noconfirm python-jinja)
(cd python-yaml; rm -rf pkg/ src/; rm *.pkg.tar.xz; pacman -Rs --noconfirm python-yaml)
(cd cython; rm -rf pkg/ src/; rm *.pkg.tar.xz; pacman -Rs --noconfirm cython)

(cd cython; makepkg -si --noconfirm)
(cd python-yaml; makepkg -si --noconfirm)
(cd python-jinja; makepkg -si --noconfirm)
(cd python-wcwidth; makepkg -si --noconfirm)
(cd ansible; makepkg -si --noconfirm)

After that

Alexandre Ferreira@ALEXWINMACHINE MSYS /c/Users/Alexandre Ferreira/ProgrammingFiles/MSYS2-packages
$ ansible --version
ansible 2.9.7
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/home/Alexandre Ferreira/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python3.8/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 3.8.3 (default, May 18 2020, 08:39:23) [GCC 9.3.0]

@lazka
Copy link
Member

lazka commented May 28, 2020

The CI errors have been fixed in master (rebasing should help). Sorry :/

@alexjorgef
Copy link
Contributor Author

The CI errors have been fixed in master (rebasing should help). Sorry :/

No problem, so far I've been using ansible without problems. And now ci is fixed, thank you

ansible/PKGBUILD Outdated Show resolved Hide resolved
ansible/PKGBUILD Outdated Show resolved Hide resolved
python-jinja/PKGBUILD Outdated Show resolved Hide resolved
python-wcwidth/PKGBUILD Outdated Show resolved Hide resolved
python-yaml/PKGBUILD Outdated Show resolved Hide resolved
@Alexpux
Copy link
Member

Alexpux commented Jun 6, 2020

@Alex1a ping

@alexjorgef alexjorgef changed the title package: add ansible pkg Add ansible 2.9.7 Jun 7, 2020
python-jinja/PKGBUILD Outdated Show resolved Hide resolved
python-wcwidth/PKGBUILD Outdated Show resolved Hide resolved
python-yaml/PKGBUILD Outdated Show resolved Hide resolved
cython/PKGBUILD Outdated Show resolved Hide resolved
cython/PKGBUILD Outdated Show resolved Hide resolved
cython/PKGBUILD Outdated Show resolved Hide resolved
python-jinja/PKGBUILD Outdated Show resolved Hide resolved
python-wcwidth/PKGBUILD Outdated Show resolved Hide resolved
python-yaml/PKGBUILD Outdated Show resolved Hide resolved
python-yaml/PKGBUILD Outdated Show resolved Hide resolved
@Alexpux
Copy link
Member

Alexpux commented Jun 15, 2020

@Alex1a fix other comments

…tarball symbolic links; edit some wrong installation directories
@Alexpux Alexpux merged commit 5d8f590 into msys2:master Jun 23, 2020
@alexjorgef alexjorgef deleted the feature/ansible-pkg branch June 10, 2022 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package Pull request which adds new package(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants