- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Feb 21 2020
Jan 25 2020
Jan 24 2020
Jan 9 2020
Dec 23 2019
Dec 22 2019
Nov 28 2019
Nov 24 2019
Nov 22 2019
Nov 15 2019
Nov 12 2019
Nov 6 2019
Oct 30 2019
Oct 29 2019
Oct 25 2019
Oct 20 2019
Oct 18 2019
Oct 16 2019
I see, i shouldn't have written "changed package". Like araujo stated, meta-data are not considered as part of the package. PORTREVISION bumps will rebuild the software from which the package is build. As long as the software contained (!) in the "package" does not change, such a bump is not needed.
In D22051#481655, @araujo wrote:You don't need the gratuitous PORTREVISION bump here!
Oct 12 2019
In D22000#480687, @dmgk wrote:In D22000#480674, @tz wrote:Also there is no need to separate the commits between updates and bumping the PORTREVISION. This can be done in one commit. So please commit this and D22001 together! :)
Thanks for approving. I was told before to split updates like this to two separate commits - https://reviews.freebsd.org/D21716#473777 :)
See my commit in D22000
Also there is no need to separate the commits between updates and bumping the PORTREVISION. This can be done in one commit. So please commit this and D22001 together! :)
Oct 10 2019
Oct 8 2019
Oct 7 2019
I'm sorry, i missed your update. Its fine now, please go ahead and thank you! :)
Oct 5 2019
In short: why? The commit message only state what the log says. But it does not tell us about the reasons. Why don't you want to install the test and the doc? I mean: normally this would be a sign of quality. Why not here?
I think the diff is fine, but the commit message needs to be improved
If possible please add the changelog or a link to the changelog of the update!
Oct 3 2019
Sep 29 2019
Sep 27 2019
In D21819#476324, @araujo wrote:In D21819#476317, @dmgk wrote:Looks like svn ate "Changes:" line in the commit message, sorry about that. Is it some kind a reserved keyword? Should I use "Changelog" or similar for itemized changes list?
I felt strange after I doubled checked the svn log, I'm not aware about reserved keywords. Usually I use changelog, but we should have no limitations. @tz are you aware of any keywords or similar limitations?
Sep 25 2019
Sep 24 2019
Beside my comment, everything looks fine. Approved :)
Sep 22 2019
In D21744#474777, @dmgk wrote:In D21744#474762, @tz wrote:Aloha Dmitri,
two short questions:
poudriere bulk -C -j120a: OK
Why only 12? Please always check both versions. Sometimes there are some side effects in building, nobody could foresee.
Hi Torsten,
Given that these ports are building fine now and there's no functional change to the software (no version updates etc) and only changes to the ports, I felt that it would be enough to test these patches only on a single FreeBSD version. I can run the same poudriere bulk builds on say 11.3 i386 and update test plan with results.
Aloha Dmitri,
Sep 21 2019
Sep 20 2019
In D21721#474028, @dmgk wrote:In D21721#473929, @araujo wrote:As you are working on this already for a while, how about ports that you are not a maintainer? Maybe an exp-run following with a portmgr@ approval would be interesting to clean up the rest of the ports.
That was my plan actually, I was just not sure what would be the best way to get something like that reviewed/committed. The complete diff is about 250KB, I was thinking about maybe breaking it to 3-4 parts for easier review. Would that make sense or it would be better to post it in a single review?
In D21349#473998, @joneum wrote:There a wrong MAINTAINER lines, for example www/mod_php74:
MAINTAINER= ports@FreeBSD.org
I think, this Lines are needless?
In D21721#473887, @dmgk wrote:In D21721#473870, @tz wrote:Since i am very unfamiliar with the go ports, i have problems to understand this diff.
In devel/awless only do-build and do-install is removed, but i do not see any usage of GO_TARGET. Same for some other ports.r505321 converted all ports to USES=go which already provides do-build and do-install targets. During that conversion, ports that were defining their own build/install targets were left alone to be cleaned up later. This commit does this cleanup for the ports that I maintain.
Is there any reason why removing the unneeded build-steps and the switch to GO_TARGET are in the same commit?
Removing targets doesn't remove any build steps, just switches the port to use default targets that go.mk provides.
GO_TARGET is used by go.mk in do-build and do-install, it doesn't make sense to set it with custom targets and removing custom targets w/o setting GO_TARGET would break the build.
Sep 19 2019
Since i am very unfamiliar with the go ports, i have problems to understand this diff.
In devel/awless only do-build and do-install is removed, but i do not see any usage of GO_TARGET. Same for some other ports.
Sep 18 2019
I'm sorry to bother you @ale , but without your approval i am not allowed to commit the needed change to php.mk. Can you please have a look at it?
Sep 16 2019
This revision is from last year and i'm pretty sure you already fixed the issue @joneum , right?
In this case let us close the review.
Please notice: the hash module is gone in PHP 7.4. Its part of the core and can't be disabled anymore. You find details here:
https://reviews.freebsd.org/D21349
Approved
Approved! Please go ahead a commit the news about you! :)
Sep 10 2019
Add new module devel/php74-ffi and fix net-mgmt Makefile.
Fix another plist issue with the php-fpm config
Sep 9 2019
Sep 8 2019
Thank you @tobik , this works! And it was the last piece. So everything is compiling and running now! :)
Sep 7 2019
@tobik Thank you, this works with RC 1 too! :)
Sep 6 2019
@ale Can you please review the changes of php.mk?
I would like to commit the RC1 without the broken modules imap and gd. I do not want to wait until the end and miss the interval to have runtime testing with an RC.