Opened 16 years ago
Closed 16 years ago
#2504 closed task (fixed)
rsync-3.0.2
| Reported by: | Ag. Hatzimanikas | Owned by: | Randy McMurchy |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.3 |
| Component: | BOOK | Version: | SVN |
| Severity: | normal | Keywords: | |
| Cc: |
Description
Version increment to 3.0.1, with many bug fixes and a few minor enhancements. In my opinion, the update should go into 6.3, please see the changelog at:
http://rsync.samba.org/ftp/rsync/rsync-3.0.1-NEWS
A couple of notes.
Technically popt is a required dependency, although if there isn't a system pop (installation) available, rsync will be happily use its bundled popt.
Now, lets say that we leave popt as an optional dependency, then we should use the "--with-included-popt" configure switch. If we'll move popt to required, then we should toggle this switch, i.e., "--without-included-popt".
And just for the shake of completeness, yodl [1] is an optional dependency, although is dead upstream and superseded than texinfo, so I wouldn't even thought to put it in the book.
Attachments (1)
Change History (7)
comment:1 by , 16 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
| Summary: | Rsync-3.0.1 → rsync-3.0.1 |
comment:2 by , 16 years ago
| Summary: | rsync-3.0.1 → rsync-3.0.2 |
|---|
Now 3.0.2, that fixes a potential buffer overflow in the xattr code.
comment:3 by , 16 years ago
"A work-around would be to disable xattrs functionality from the /etc/rsyncd.conf configuration file." [ from one report I saw on the issue. ] Which might be a good idea to recommend before 6.3 is released and the new version of rsync is in the book.
comment:4 by , 16 years ago
As far as I have seen the popt library stay as optional. I renamed the installed popt libs and rsync will use the own ones. Renamed the libs back to the right names, rsync used that ones. So i think the popt depedency is good as it is optional. The --with-included-popt forces rsync to use its own popt library even the popt is installed system wide.
comment:5 by , 16 years ago
I committed the update without reviewing this ticket. I'll go back and make the popt thing right.
comment:6 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
After actually looking at the configure.in script and doing real-time testing, I don't agree with the initial observation that something needs to be done with the dependency.
If popt is not installed, rsync uses internal copy without problems or issues.
If popt is installed, it uses the system-installed copy without problems or issues.
No switches passed to configure required in either case.
Yes, something needs to be done for the popt dependency. I'm in favor of keeping it optional and using the "with-system-popt" (or whatever it is) if it won't build without it.
I'll check it out and do the update.